Skip to content
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

Pruning #3872

Merged
merged 3 commits into from
Oct 3, 2020
Merged

Pruning #3872

merged 3 commits into from
Oct 3, 2020

Conversation

wtracy
Copy link
Contributor

@wtracy wtracy commented Oct 2, 2020

This changeset prunes some unnecessary code (probably leftover from past refactors):

  • A unnecessary conversion to uppercase in misc.py on a value that is simply tested for existence and then discarded
  • A duplicate definition of OPTION_RGX in constants.py

…pose as the list is simply tested with set() and then discarded.
… else. Removing this declaration, as it serves no purpose.
# Allow stopping after the first semicolon/hash encountered,
# so that an option can be continued with the reasons
# why it is active or disabled.
OPTION_RGX = re.compile(r"\s*#.*\bpylint:\s*([^;#]+)[;#]{0,1}")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The re module is then useless here. It's the reason the CI doesn't success.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wtracy thanks for this PR. I'am ok with it as far as the CI finishes successfully.

@coveralls
Copy link

coveralls commented Oct 3, 2020

Coverage Status

Coverage decreased (-0.01%) to 90.71% when pulling c15d3f0 on wtracy:pruning into 548b38e on PyCQA:master.

@hippo91 hippo91 merged commit 98d2c06 into pylint-dev:master Oct 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants