-
Notifications
You must be signed in to change notification settings - Fork 156
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
Decorator name regression #284
Conversation
Codecov Report
@@ Coverage Diff @@
## main #284 +/- ##
=======================================
Coverage 98.96% 98.96%
=======================================
Files 21 21
Lines 676 679 +3
=======================================
+ Hits 669 672 +3
Misses 7 7
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
vulture/noqa.py
Outdated
@@ -3,7 +3,7 @@ | |||
|
|||
NOQA_REGEXP = re.compile( | |||
# Use the same regex as flake8 does. | |||
# https://gitlab.com/pycqa/flake8/-/tree/master/src/flake8/defaults.py | |||
# https://gitlab.com/pycqa/flake8/-/tree/main/src/flake8/defaults.py |
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.
flake8
moved to GitHub last year. Let's use the new URL instead?
# https://gitlab.com/pycqa/flake8/-/tree/main/src/flake8/defaults.py | |
# https://github.com/PyCQA/flake8/blob/main/src/flake8/defaults.py |
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.
Oops, looks like these changes are only appearing here because GitHub wasn't able to rebase this branch (probably because of the master
-> main
rename). Changes already went in fb126c7. I'll send a PR separately.
This PR is probably redundant because #285 tackles the same problem. |
No description provided.