New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add overrides
option to config (#3128)
#5521
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hudochenkov This is amazing, thank you!
The code and tests look good to me.
I've suggested some minor tweaks to the documentation and error message wording.
Many thanks again! This is an exciting addition that gets us closer to 14.0.0.
Co-authored-by: Richard Hallows <jeddy3@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fantastic, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hudochenkov Thanks, this is awesome work! 😄
I've left some comments, but I think this PR is mostly fine!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hudochenkov Thank you! LGTM! 👏🏼
|
I'll create an issue so that we can discuss what can we do about this. |
Closes #3128.
It took quite some time to figure out how config resolutions works. Codebase is very tangled in my opinion :(
For every linted file we produce new config from initial config. All overrides are resolved and applied on a config and file gets normal config with no
overrides
property.I took
overrides
docs from ESLint documentation with some modifications.Interesting thing that all properties that regular config support, are supported by overrides. So when we #5142 is done, all options would be working automatically for
overrides
.