-
Notifications
You must be signed in to change notification settings - Fork 654
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
Update ComparisonToEmptyStringRule.py #770
Conversation
The conditionals with bare variables will be deprecated. It's causing a lint warning atm. See CONDITIONAL_BARE_VARS https://docs.ansible.com/ansible/latest/reference_appendices/config.html#conditional-bare-vars
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.
fair enough
While the change is ok, I cannot accept it like this because you should also update the docs so that the CI doesn't fail: https://github.com/ansible/ansible-lint/pull/770/checks?check_run_id=652398319#step:10:53. |
Thank you for the review. I've updated the docs and submitted PR #772 Will you be able to squash and merge #770 and #772 ? |
Just update this PR, it'll only be green if they are together. |
@vbotka Please close the doc update and include it here. This can be avoided by always running |
Done. "tox -e lint" passed before the push. Thank you for comments. |
Looks green to me |
The conditionals with bare variables will be deprecated. It's causing a lint warning atm. See CONDITIONAL_BARE_VARS https://docs.ansible.com/ansible/latest/reference_appendices/config.html#conditional-bare-vars