-
Notifications
You must be signed in to change notification settings - Fork 158
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 for linting and formatting, update CI #978
Conversation
7c3392d
to
2cdca6b
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #978 +/- ##
==========================================
+ Coverage 76.17% 76.28% +0.10%
==========================================
Files 83 83
Lines 6406 6384 -22
==========================================
- Hits 4880 4870 -10
+ Misses 1526 1514 -12
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
cfd5849
to
32978c9
Compare
dbe58f5
to
0df847d
Compare
0df847d
to
c83e095
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.
Thanks a lot! I did not go through tests and tutorials, but left some comments on the rest.
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.
Thanks!
afe00fc
to
246c8b0
Compare
Main changes
black
,isort
andflake
byruff
inpyproject.toml
pre-commit
hookspre-commit
hooksruff
errors so that we can introduceruff
linting and formatting in pre-commit hooks for the hackathonWhy are there so many changed files?
pre-commit
fixes: trailing whitespaces and empty linesruff check --fix
tosbi/
,examples/
tutorials/
tests/
ruff format .
ruff check .
errors manually 😵fixes #934
fixes #959
fixes #960
Notes for reviewing
As there are so many file changes, I recommend reviewing the individual commits separately. I tried to group changes into specific commits.