-
Notifications
You must be signed in to change notification settings - Fork 668
chore: add isort as a checker #1463
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1463 +/- ##
==========================================
+ Coverage 90.83% 90.85% +0.02%
==========================================
Files 73 73
Lines 4027 4037 +10
==========================================
+ Hits 3658 3668 +10
Misses 369 369
Flags with carried forward coverage won't be shown. Click here to find out more.
|
* Add an isort tox environment * Run the isort tox environment using --check in the Github CI https://pycqa.github.io/isort/
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, finally don't need to worry about whether the first 10 lines in new files make sense 😁 Just have a few thoughts.
Fix the import ordering using isort. https://pycqa.github.io/isort/
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.
Awesome! Thanks again :)
isort is a tool to make sure that imports are ordered correctly.
Add a tox environment for it and also add it to the Github workflow to
ensure it is checked for Pull Requests.
Run isort on all the code and commit changes it made.
https://pycqa.github.io/isort/