-
Notifications
You must be signed in to change notification settings - Fork 39
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
Adds unused variable rule (F841
) to ruff
#3035
Conversation
Quality Gate passedIssues Measures |
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
Test results 9 files 9 suites 8m 14s ⏱️ Results for commit d581a81. ♻️ This comment has been updated with latest results. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3035 +/- ##
=======================================
Coverage 60.33% 60.33%
=======================================
Files 606 606
Lines 43700 43700
Branches 48 48
=======================================
Hits 26366 26366
Misses 17322 17322
Partials 12 12 ☔ View full report in Codecov by Sentry. |
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.
The MegaLinter check is failing. Maybe this needs to be rebased to the latest master first?
And yeah, apparently I didn't see the big red blocked-label, so I guess my comment is obvious ;) |
73971f4
to
c4931f9
Compare
c4931f9
to
bad52cc
Compare
bad52cc
to
c72208e
Compare
c72208e
to
d581a81
Compare
Quality Gate passedIssues Measures |
As #3026, up for discussion. This also adds
F842
- Local variable {name} is annotated but never used.