chore(linter): Remove one last reference to the regexp plugin.#16772
chore(linter): Remove one last reference to the regexp plugin.#16772graphite-app[bot] merged 1 commit intomainfrom
Conversation
There was a problem hiding this comment.
No issues found in the modified code. The change cleanly removes the final regexp plugin reference and falls back to the default _ => plugin_name behavior without introducing complexity or behavioral ambiguity beyond the intended removal.
Summary of changes
What changed
- Removed the last remaining mapping for the
regexpplugin inplugin_name_to_prefix()by deleting the match arm:"regexp" => "eslint-plugin-regexp"
Files touched
crates/oxc_linter/src/context/mod.rs
There was a problem hiding this comment.
Pull request overview
This PR removes the last remaining reference to the eslint-plugin-regexp plugin from the codebase. The plugin_name_to_prefix function previously mapped the short plugin name "regexp" to "eslint-plugin-regexp", but since the regexp plugin has been completely removed from the linter (no rules directory exists for it), this mapping is no longer needed.
Key Changes:
- Removed the "regexp" case from the
plugin_name_to_prefixfunction's match statement
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
CodSpeed Performance ReportMerging #16772 will not alter performanceComparing Summary
Footnotes
|
Merge activity
|
Eliminate one more reference that was still lingering.
e380b35 to
fbcb49a
Compare
Eliminate one more reference that was still lingering.