-
Notifications
You must be signed in to change notification settings - Fork 640
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 opt-in rule to check whether FQCN is used for builtins #1614
Conversation
…an ansible module that was used
…le is never ignored
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 like the rule (for the playbooks I help maintain, we use FQCNs in all tasks). But this rule should definitely be disabled by default because as far as I know, there is no plan to deprecate bare name usage.
I did add a few nitpicks, but apart from that, the rule looks good to me.
Thanks for your review @tadeboro, I adapted the code to include your comments |
Co-authored-by: Felix Fontein <felix@fontein.de>
As I have written in #1587, we currently have a regression for our custom rule that checks whether FQCNs are used everywhere.
As @ssbarnea suggested there, I added this rule upstream and made it opt-in. To not break current pipelines, it is by default turned off in the CI. I needed to change the
rule_runner
fixture, such that this rule is ran for therule_runner
nevertheless (this also seems like the more sensible default to me, as you include a specific rule there and expect it to be run).This is my first contribution to this project, so feedback is welcome