-
Notifications
You must be signed in to change notification settings - Fork 187
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
setup.py: Drop flake8-polyfill dependency #186
Conversation
Newer versions (> 3.x) of flake8 no longer require flake8-polyfill
pep8-naming/src/pep8ext_naming.py Line 10 in 407fa0c
pep8-naming/src/pep8ext_naming.py Lines 143 to 171 in 407fa0c
|
oops. my bad. didn't really check where it is used. edit: It looks non-trivial. Do you have any suggestions on how do I go about removing this dep? |
The options need to be registered directly with |
#199 is my take on removing this dependency. |
Closed by #199 |
Newer versions (> 3.x) of flake8 no longer require flake8-polyfill
Fixes #185
Also refer PyCQA/flake8-polyfill#14