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
Extend fqcn rule to complain on collections
keyword
#2825
Conversation
This is awesome. TY |
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.
A test would be cool here
81ac8b2
to
416afde
Compare
@cidrblock thanks for the feedback. I've changed the message based on it and extended the test case to also cover this. |
416afde
to
fa79abd
Compare
Rebased on top of main to re-run CI. The failures weren't related to the PR, just timeouts when talking to galaxy.ansible.com. Would it be possible to add some retries to those calls, so that a single timeout doesn't fail a whole CI job? |
@jcgruenhage I know that galaxy unreliability is a PITA and I already did some work to reduce its use, still I was not able to fully de-couple it, especially as it is impossible to install roles offline. |
Well, but CI passed this time, so it's all good I suppose :D For merging, it's missing those two pending reviews I'd assume? |
@ssbarnea anything still needed here? I'd like to see this merged at some point. |
collections
keywordcollections
keyword
62231aa
to
7c2a35a
Compare
I had opened a discussion about this a month ago (#2728), but since there's no interaction happening there I'm opening a PR.