Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Update cli_parse plugins and testcase #182

Merged
merged 16 commits into from
Feb 22, 2021

Conversation

ganeshrn
Copy link
Member

@ganeshrn ganeshrn commented Dec 1, 2020

SUMMARY

Depends-On: ansible-collections/ansible.utils#33
Depends-On: ansible/ansible-zuul-jobs#731
Depends-On: #223

  • Remove cli_parse action plugin as it will reside in ansible.utils collection going forward and add
    redirection and deprecation notice
  • Update test cases
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

cli_parse

ADDITIONAL INFORMATION

galaxy.yml Outdated Show resolved Hide resolved
@samccann
Copy link
Contributor

samccann commented Dec 1, 2020

Should this have a changelog fragment?

@ganeshrn
Copy link
Member Author

ganeshrn commented Dec 3, 2020

recheck

1 similar comment
@pabelanger
Copy link
Contributor

recheck

@Qalthos Qalthos added the bug This issue/PR relates to a bug. label Dec 3, 2020
Copy link
Contributor

@samccann samccann left a comment

Choose a reason for hiding this comment

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

docs portion lgtm

@ganeshrn ganeshrn force-pushed the cli_parse_move branch 2 times, most recently from f4de92f to 942e07a Compare December 10, 2020 08:50
meta/runtime.yml Outdated Show resolved Hide resolved
@ganeshrn ganeshrn added the blocks_release Ensure this is merged before a release is made label Jan 5, 2021
@Qalthos Qalthos added mergeit Gate PR in Zuul CI and removed mergeit Gate PR in Zuul CI labels Jan 22, 2021
@ganeshrn
Copy link
Member Author

ganeshrn commented Jan 25, 2021

The PR is for 2.0.0 release

@Qalthos Qalthos removed blocks_release Ensure this is merged before a release is made mergeit Gate PR in Zuul CI labels Jan 25, 2021
@ganeshrn
Copy link
Member Author

ganeshrn commented Feb 2, 2021

recheck

@ganeshrn ganeshrn added the blocks_release Ensure this is merged before a release is made label Feb 3, 2021
@ganeshrn
Copy link
Member Author

ganeshrn commented Feb 5, 2021

recheck

@pabelanger
Copy link
Contributor

recheck

@pabelanger
Copy link
Contributor

recheck

@ganeshrn ganeshrn added the mergeit Gate PR in Zuul CI label Feb 22, 2021
@ganeshrn
Copy link
Member Author

recheck

@ganeshrn ganeshrn added mergeit Gate PR in Zuul CI and removed mergeit Gate PR in Zuul CI labels Feb 22, 2021
@ansible-zuul ansible-zuul bot merged commit 12c98d8 into ansible-collections:main Feb 22, 2021
Qalthos pushed a commit to Qalthos/ansible.netcommon that referenced this pull request Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocks_release Ensure this is merged before a release is made bug This issue/PR relates to a bug. mergeit Gate PR in Zuul CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants