Skip to content

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

Merged
merged 2 commits into from
May 25, 2021
Merged

chore: add isort as a checker #1463

merged 2 commits into from
May 25, 2021

Conversation

JohnVillalovos
Copy link
Member

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/

@codecov-commenter
Copy link

codecov-commenter commented May 22, 2021

Codecov Report

Merging #1463 (4a505df) into master (1508eb7) will increase coverage by 0.02%.
The diff coverage is 98.14%.

❗ Current head 4a505df differs from pull request most recent head 4cfd7b7. Consider uploading reports for the commit 4cfd7b7 to get more accurate results

@@            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              
Flag Coverage Δ
cli_func_v4 80.40% <98.14%> (+0.04%) ⬆️
py_func_v4 79.48% <96.29%> (+0.05%) ⬆️
unit 81.94% <98.14%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
gitlab/__main__.py 0.00% <ø> (ø)
gitlab/const.py 100.00% <ø> (ø)
gitlab/v4/objects/access_requests.py 100.00% <ø> (ø)
gitlab/v4/objects/appearance.py 100.00% <ø> (ø)
gitlab/v4/objects/award_emojis.py 100.00% <ø> (ø)
gitlab/v4/objects/badges.py 100.00% <ø> (ø)
gitlab/v4/objects/boards.py 100.00% <ø> (ø)
gitlab/v4/objects/branches.py 73.52% <ø> (ø)
gitlab/v4/objects/broadcast_messages.py 100.00% <ø> (ø)
gitlab/v4/objects/container_registry.py 83.33% <ø> (ø)
... and 57 more

@JohnVillalovos JohnVillalovos changed the title Add isort as a checker chore: add isort as a checker May 22, 2021
@JohnVillalovos JohnVillalovos requested a review from nejch May 22, 2021 15:49
  * Add an isort tox environment
  * Run the isort tox environment using --check in the Github CI

https://pycqa.github.io/isort/
@JohnVillalovos JohnVillalovos self-assigned this May 22, 2021
Copy link
Member

@nejch nejch left a 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.

Copy link
Member

@nejch nejch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Thanks again :)

@nejch nejch merged commit 7824811 into python-gitlab:master May 25, 2021
@JohnVillalovos JohnVillalovos deleted the jlvillal/isort branch May 26, 2021 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants