-
Notifications
You must be signed in to change notification settings - Fork 660
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
name[casing]
error should be raised for notify
task param
#4035
Comments
name[casing]
issue should be raised for "notify" field under a task
name[casing]
issue should be raised for "notify" field under a taskname[casing]
issue should be raised for "notify" task param
name[casing]
issue should be raised for "notify" task paramname[casing]
error should be raised for "notify" task param
name[casing]
error should be raised for "notify" task paramname[casing]
error should be raised for notify
task param
The become.yml playbook had its name[casing] violation fixed because said playbook isn't expected to be tested for name[casing] violations.
This produces false errors for cases where a handler uses an explicit E.g. this handler:
in combination with this task:
will trigger
even though both task and handler names start with uppercase characters. |
I don't like this one. I usually stick to simple keywords when notifying handlers which I define under Checking the casting of the handler task name should be enough, it should catch any task names. No need to do it for the |
When we raise a
name[casing]
issue for a task under handler, the same should be raised for a task which is calling this handler task usingnotify
task parameterRelated #4028
The text was updated successfully, but these errors were encountered: