-
Notifications
You must be signed in to change notification settings - Fork 530
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
STY: Apply ruff/Perflint rules (PERF) #3674
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3674 +/- ##
==========================================
+ Coverage 70.84% 70.86% +0.02%
==========================================
Files 1276 1276
Lines 59314 59229 -85
Branches 9826 9812 -14
==========================================
- Hits 42019 41975 -44
+ Misses 16119 16079 -40
+ Partials 1176 1175 -1 ☔ View full report in Codecov by Sentry. |
0befcf4
to
7af729f
Compare
7af729f
to
b7da9dc
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.
Partial review
8b17632
to
0e6a385
Compare
PERF401 Use a list comprehension to create a transformed list
PERF403 Use a dictionary comprehension instead of a for-loop
63517ff
to
d5edcc9
Compare
Co-authored-by: Chris Markiewicz <effigies@gmail.com>
d5edcc9
to
60aa048
Compare
Summary
Apply Perflint (PERF) rules, except: