-
-
Notifications
You must be signed in to change notification settings - Fork 595
feat(linter): add fix for unused disable directive #11708
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
feat(linter): add fix for unused disable directive #11708
Conversation
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
CodSpeed Instrumentation Performance ReportMerging #11708 will not alter performanceComparing Summary
|
94f8045
to
97b748b
Compare
3e64d96
to
d7fa8df
Compare
d7fa8df
to
8884040
Compare
ccd4538
to
2f9d125
Compare
2f9d125
to
b3cd1ee
Compare
285f0a5
to
b39d1fa
Compare
b3cd1ee
to
c20a77d
Compare
c20a77d
to
b0267e6
Compare
8affa74
to
e2fbd4c
Compare
Merge activity
|
Added tests for later, see linked issue #11709.  ``` // oxlint-disable-next-line no-debugger -- on wrong line (() => { debugger; })(); ``` will fix to: ``` // (() => { debugger; })(); ``` Not the best, but at least a start. Don't know what kind of fix it should be. It is currently a `SafeFix`. ToDo: - No other code actions ✔️ - generate correct spans for directives with multiple rules ✔️
e2fbd4c
to
1a54184
Compare
Added tests for later, see linked issue #11709.
will fix to:
Not the best, but at least a start.
Don't know what kind of fix it should be. It is currently a
SafeFix
.ToDo: