-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
pylint plugin to catch due/past-due deprecated calls #44143
Conversation
This comment has been minimized.
This comment has been minimized.
class AnsibleDeprecatedChecker(BaseChecker): | ||
"""Checks string formatting operations to ensure that the format string | ||
is valid and the arguments match the format string. | ||
""" |
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.
This docstring needs to be updated.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
65dab3d
to
b8181b6
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
b8181b6
to
45eff20
Compare
18c963b
to
a81500f
Compare
Removed |
This comment has been minimized.
This comment has been minimized.
The test
|
This reverts commit 4e84034.
Ok, now that I have logged all deprecations for 2.8, I've added the ignore back in. |
@mattclay @nitzmahone @abadger @ryansb Is there anything additional you would like to see done here? Everything is passing right now (rebuilding right now due to stale_ci), and we will leave |
Can we make the flip automated by checking the base branch and flipping it on if there is |
We need to keep tests passing on the devel branch, so we shouldn't change the version being checked automatically. Running the test needs to be done as part of the release process when we update the version number. The test should be updated, and the necessary fixes applied as a PR. |
I've just added a script I used to create issues for the deprecated items found by this pylint plugin. Run something like:
Where |
@ryansb, the check needs to run on the devel branch after the previous version has its stable-X.Y branch created. So the enhancement you propose would be testing something different. |
The test
|
SUMMARY
pylint plugin to catch due/past-due deprecated calls
ISSUE TYPE
COMPONENT NAME
test/sanity/pylint/plugins/deprecated.py
ANSIBLE VERSION
ADDITIONAL INFORMATION