-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add flake8 plugin pep8-naming as development dependency #40
Conversation
ab27100
to
b341574
Compare
b341574
to
23c95b1
Compare
1550671
to
dd431c8
Compare
dd431c8
to
03a115f
Compare
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.
I made some suggestions. One further comment: it would be nice if you could split the commits in one (or more) adding the functionality and one (or more) changing the code or applying fixes to the issues.
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.
Please be aware of my comment in the last review: #40 (review)
9f12912
to
e14a68a
Compare
Description
indicators/mapping-saturation/
New or changed dependencies
Checklist
main
(e.g. throughgit rebase main
)[ ] My code follows the style guide and was checked with pre-commit before committing[ ] I have commented my code[ ] I have added sufficient unit and integration tests