Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: use ruff instead of flake8 and isort #699

Merged
merged 6 commits into from
Jan 11, 2023
Merged

chore: use ruff instead of flake8 and isort #699

merged 6 commits into from
Jan 11, 2023

Conversation

12rambau
Copy link
Member

@12rambau 12rambau commented Jan 10, 2023

ruff is 100x faster and natively compatible with Black

  • drop of flake8 and isort
  • solve the 300+ issues raised by ruff
  • solve the circular import issues

Fix #694

@12rambau 12rambau marked this pull request as ready for review January 10, 2023 21:19
@codecov
Copy link

codecov bot commented Jan 10, 2023

Codecov Report

Merging #699 (9832fa0) into main (6b3f6e8) will increase coverage by 0.01%.
The diff coverage is 93.75%.

@@            Coverage Diff             @@
##             main     #699      +/-   ##
==========================================
+ Coverage   93.60%   93.62%   +0.01%     
==========================================
  Files          39       39              
  Lines        4332     4328       -4     
==========================================
- Hits         4055     4052       -3     
+ Misses        277      276       -1     
Impacted Files Coverage Δ
sepal_ui/aoi/aoi_model.py 89.14% <ø> (ø)
sepal_ui/aoi/aoi_tile.py 100.00% <ø> (ø)
sepal_ui/aoi/aoi_view.py 98.87% <ø> (ø)
sepal_ui/conf.py 100.00% <ø> (ø)
sepal_ui/frontend/resize_trigger.py 100.00% <ø> (ø)
sepal_ui/mapping/aoi_control.py 100.00% <ø> (ø)
sepal_ui/mapping/draw_control.py 97.61% <ø> (ø)
sepal_ui/mapping/fullscreen_control.py 100.00% <ø> (ø)
sepal_ui/mapping/layer.py 100.00% <ø> (ø)
sepal_ui/mapping/layer_state_control.py 98.00% <ø> (ø)
... and 29 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@12rambau 12rambau merged commit e441e2d into main Jan 11, 2023
@12rambau 12rambau deleted the ruff branch January 11, 2023 08:50
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

move to ruff
1 participant