Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Adding ansible --tag handling #33

Merged
merged 1 commit into from
Feb 10, 2022
Merged

Adding ansible --tag handling #33

merged 1 commit into from
Feb 10, 2022

Conversation

LegenJCdary
Copy link
Collaborator

No description provided.

@LegenJCdary LegenJCdary marked this pull request as draft February 9, 2022 14:38
@LegenJCdary LegenJCdary force-pushed the add_tag_handling branch 2 times, most recently from ed1a12f to e3a153c Compare February 9, 2022 20:37
@LegenJCdary LegenJCdary marked this pull request as ready for review February 9, 2022 22:51
@@ -65,3 +65,7 @@ tasks:
allow_limit:
type: boolean
required: True
tags:
type: list
required: True
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it should be required. My expectation is that it should be used rarely.

Copy link
Owner

@cinek810 cinek810 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just make it non required

@LegenJCdary LegenJCdary merged commit e82d82f into main Feb 10, 2022
@LegenJCdary LegenJCdary deleted the add_tag_handling branch February 10, 2022 09:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants