#53925: build: cleanup eslint/oxlint disable comments
Merged
Description of Change
We had a handful of stale eslint-disable comments so turn on reportUnusedDisableDirectives and clean them up. Some like no-undef were rules that got switched off in the migration to oxlint that would be nice to turn back on, but require additional configuration/cleanup to do so.
This also renames all eslint-disable to oxlint-disable to accurately reflect that we use oxlint now.
Checklist
- I have built and tested this change
- I have filled out the PR description
Release Notes
Notes: none
Backports
45-x-y
PendingWaiting for a manual backport
Semver Impact
Major
Breaking changes
Minor
New features
Patch
Bug fixes
None
Docs, tests, etc.
Semantic Versioning helps users understand the impact of updates:
- Major (X.y.z): Breaking changes that may require code modifications
- Minor (x.Y.z): New features that maintain backward compatibility
- Patch (x.y.Z): Bug fixes that don't change the API
- None: Changes that don't affect using facing parts of Electron