Skip to content
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

Use ruff isort plugin instead of isort #321

Merged
merged 1 commit into from
Nov 5, 2023
Merged

Use ruff isort plugin instead of isort #321

merged 1 commit into from
Nov 5, 2023

Conversation

farmio
Copy link
Member

@farmio farmio commented Nov 5, 2023

  • Use ruff isort plugin instead of isort
  • Move ruff to separate tox env (so dependabot updates can trigger a run - which doesn't work for update pre-commit hoos script)
  • Run ruff in pre-commit only on pre-commit
  • Remove isort from requirements
  • Remove black target_version as it is now inferred from project.requires-python

Copy link

codecov bot commented Nov 5, 2023

Codecov Report

Merging #321 (95d81f3) into main (fc57075) will increase coverage by 0.02%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #321      +/-   ##
==========================================
+ Coverage   97.48%   97.50%   +0.02%     
==========================================
  Files          21       21              
  Lines        1073     1082       +9     
==========================================
+ Hits         1046     1055       +9     
  Misses         27       27              

see 3 files with indirect coverage changes

@farmio farmio merged commit 380be02 into main Nov 5, 2023
@farmio farmio deleted the ruff-isort branch November 5, 2023 09:09
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.

1 participant