-
Notifications
You must be signed in to change notification settings - Fork 156
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
Apply more ruff rules #633
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #633 +/- ##
==========================================
- Coverage 78.25% 74.44% -3.82%
==========================================
Files 14 14
Lines 1122 1119 -3
==========================================
- Hits 878 833 -45
- Misses 244 286 +42 ☔ View full report in Codecov by Sentry. |
fdff091
to
a2bf5b9
Compare
I'm not seeing any changes to |
e52494a
to
795d5df
Compare
May I ask what prompted this change? |
9c9fa29
to
531756f
Compare
I just tried to apply some rules that raised a few valid issues in other projects. |
59869f3
to
a2c0acc
Compare
a2c0acc
to
c95b95b
Compare
PERF401 Use a list comprehension to create a transformed list
PIE810 Call `endswith` once with a `tuple`
PLC0208 Use a sequence type instead of a `set` when iterating over values
c95b95b
to
8f0fc5e
Compare
FLY002 Consider f-string instead of string join
8f0fc5e
to
517db89
Compare
517db89
to
494b33c
Compare
No description provided.