chore: upgrade xo to 2.0#8907
Conversation
Bump xo from ^1.1.0 to ^2.0.2 (ESLint 10). Suppress new rules (require-unicode-regexp, unicorn/no-immediate-mutation, etc.) that fire on legacy jQuery code not worth refactoring. Closes SickChill#8879
|
Important Review skippedIgnore keyword(s) in the title. ⛔ Ignored keywords (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
^1.1.0to^2.0.2(ESLint 10)require-unicode-regexp,unicorn/no-immediate-mutation,@stylistic/curly-newline,unicorn/no-array-sort,max-lines)yarn testpasses cleanly (6 warnings only — long lines in legacy files)Closes #8879
Notes
The suppressed rules all fire exclusively on
core.js(4500-line legacy jQuery file) which will eventually be replaced by the webpack frontend. Not worth refactoring now.