fix: remove obsolete ESLint 10 --ext option - #528
Conversation
|
@Cherry7593 is attempting to deploy a commit to the afc163's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
nrps9909
left a comment
There was a problem hiding this comment.
Verified exact head 889c86e01396764fb7aa256b8a44736a9c386773 with the ESLint 10.6 flat config in this repository. npm run lint exits successfully with 0 errors (12 existing hook/disable warnings), and npm run tsc passes. I also compared JSON lint output from eslint src/ against the previous eslint src/ --ext .ts,.tsx,.jsx,.js,.md: both discover the same 19 source files and report the same result, so removing the obsolete discovery flag does not reduce current lint coverage. No blocking issues found.
What changed
Remove the obsolete ESLint 10
--extoption from the repository lint script.Why
This repository uses ESLint 10 flat config, where
--extis no longer accepted. The flat config already defines TypeScript/React handling and ignores, so lintingsrc/preserves the same owned source scope.Validation
Development-script only; no runtime API changes.