remove options from pylint #37054
Merged
remove options from pylint #37054
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
These are going to be removed in pylint 1.6, and it doesn't appear that we are using them, and just causes extra output.
s0undt3ch
approved these changes
Oct 17, 2016
|
👍 |
gitebra
pushed a commit
to gitebra/salt
that referenced
this pull request
Oct 20, 2016
* upstream/develop: removed trailing whitespaces in apache_module.py Revert "Fix integration tests launching - remove redundant "config_dir" argument in call to ShellTestCase.run_run from SaltDaemonScriptBase._wait_until_running" use versionadded and deprecated warnings in apache_module Fix link in libvirt docs Clearer check. Add run_on_start to SCHEDULE_CONF (saltstack#37098) Fix issue saltstack#37063 Add one more paragraph Restructure grain writing docs [2016.3] Update salt.utils.cloud references to __utils__ for cache funcs remove options from pylint (saltstack#37054) Update the Fedora installation docs Move strip to powershell block, add -NoProfile Further clarification on new grains docs from saltstack#37028 Prevent source files in /tmp from being deleted by file.managed states (saltstack#37023) Prevent fallback to uptime on status.uptime for Solaris 9, 10 and 11. args does not always exist (saltstack#37019) Mitigates failure reports when making sure an existing supervisor process group is running, despite success.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
These are going to be removed in pylint 1.6, and it doesn't appear that
we are using them, and just causes extra output.
Previous behavior
This was printed at the start of each pylint run, and causes popups in IDEs that can use pylint as a plugin
New Behavior
No more Warning messages.
@s0undt3ch can you review this and make sure I am not wrong? Thanks!