-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Bug: overrideConfig requires overrideConfigFile #19279
Copy link
Copy link
Closed
Labels
acceptedThere is consensus among the team that this change meets the criteria for inclusionThere is consensus among the team that this change meets the criteria for inclusiondocumentationRelates to ESLint's documentationRelates to ESLint's documentationevaluatingThe team will evaluate this issue to decide whether it meets the criteria for inclusionThe team will evaluate this issue to decide whether it meets the criteria for inclusionrepro:yesIssues with a reproducible exampleIssues with a reproducible example
Description
Activity
Metadata
Metadata
Assignees
Labels
acceptedThere is consensus among the team that this change meets the criteria for inclusionThere is consensus among the team that this change meets the criteria for inclusiondocumentationRelates to ESLint's documentationRelates to ESLint's documentationevaluatingThe team will evaluate this issue to decide whether it meets the criteria for inclusionThe team will evaluate this issue to decide whether it meets the criteria for inclusionrepro:yesIssues with a reproducible exampleIssues with a reproducible example
Type
Projects
- StatusShow more project fieldsComplete
Environment
Environment Info:
Node version: v22.12.0
npm version: v11.0.0
Local ESLint version: v9.17.0 (Currently used)
Global ESLint version: Not found
Operating System: darwin 24.1.0
What parser are you using?
Default (Espree)
What did you do?
eslint.config.js:What did you expect to happen?
I would have expected that the
overrideConfigwould take effect, and therefore thatundefinedwould be logged as therules.What actually happened?
{ semi: 2 }is logged, until you addoverrideConfigFile: trueto the options.Link to Minimal Reproducible Example
https://github.com/abrahamguo/repro/tree/override-config
Participation
Additional comments
It's unclear to me whether this is an issue with ESLint, or whether this is intended behavior and an issue with ESLint's docs.
The docs state this:
This makes it sound like you should use either one or the other, not both together.