-
Notifications
You must be signed in to change notification settings - Fork 23.8k
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
Add format options for ansible-galaxy collection list #73474
Conversation
bdaa690
to
c735ab2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm fine with the idea, we just need a changelog fragment and integration test for this. You can find the current tests at https://github.com/ansible/ansible/blob/devel/test/integration/targets/ansible-galaxy-collection/tasks/list.yml.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs integration tests + some structural improvements:
@jborean93 @webknjaz I have addressed the majority of your feedback. However, I have been unsuccessful at getting the integration tests to work on my machine, so I cannot add a new one. Using the command:
I see the error:
I spent about an hour trying to figure this out and had 0 luck. |
These integration tests need to be run with --docker |
This is done. |
Quick, someone push the button before I break the CI again! |
SUMMARY
ISSUE TYPE
COMPONENT NAME
ansible-galaxy
ADDITIONAL INFORMATION
This would be useful for an upcoming feature in AWX/Tower.