-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Flake8 rule C901 is about McCabe code complexity #3673
Conversation
It is not about flake8-comprehensions.
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.
Nice catch, just one question.
@@ -2,7 +2,7 @@ | |||
# Exclude the grpc generated code | |||
exclude = ./manim/grpc/gen/*, __pycache__,.git, | |||
per-file-ignores = __init__.py:F401 | |||
max-complexity = 15 | |||
max-complexity = 29 |
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.
Why 29?
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.
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.
Thanks!
* Flake8 rule C901 is about McCabe code complexity It is not about flake8-comprehensions. * max-complexity = 29
Flake8 C901 is not about
flake8-comprehensions
which are C4xx rules.This misconfiguration has made us blind to increasing code complexity... https://pypi.org/project/mccabe
Related to:
Overview: What does this pull request change?
Motivation and Explanation: Why and how do your changes improve the library?
Links to added or changed documentation pages
Further Information and Comments
Reviewer Checklist