-
Notifications
You must be signed in to change notification settings - Fork 61
updating lint and galaxy files #256
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
Conversation
modified: .github/workflows/ansible-test.yml
|
I think there is a bug with ansible-compat causing this to error out. ansible/ansible-content-actions#55 |
|
trying to decide what to do here... the CI is broken but it appears to be a bug with ansible-compat. so idk if we should change CI test for now to work around it or just ignore it and force merge the change for now and hope they fix it soon |
|
I'm not a fan of "hope they fix it soon" given it's already been 2 weeks. If it's a trivial workaround to our CI so we don't have to be annoyed anymore, I would vote for that. |
swapdisk
left a comment
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.
Please can we update to deal with the ansible-compat bug or disable that check in our CI workflow until the bug gets fixed?
|
ok I know it says failed but that is because I made a change to the action but reference the one in main so once it gets updated it will work |
swapdisk
left a comment
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.
LGTM
No description provided.