[Babel 8] Ignore config files by default for eslint-parser#16871
[Babel 8] Ignore config files by default for eslint-parser#16871liuxingbaoyu wants to merge 2 commits intobabel:mainfrom
eslint-parser#16871Conversation
|
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/58524 |
|
|
||
| return getDefaultParserOptions(parseOptions); | ||
| } | ||
| if (process.env.BABEL_8_BREAKING) { |
There was a problem hiding this comment.
Why can't @babel/eslint-parser release a major without bumping other packages?
There was a problem hiding this comment.
Maybe it's just for simplicity. :)
But it may change in the future.
|
What was the motivation for this? |
|
Loading configuration files can be slow, so this improves performance over the default. |
|
Mh, in which case would somebody want to use |
|
I thought there was an agreement to change it somewhere. As one uses I just want to say there are too many This is what I use usually https://github.com/sindresorhus/eslint-plugin-unicorn/blob/eb08fc523c3229722df167d91095cf4f5f5895be/test/utils/parsers.js#L21 Search for If we don't want change the default behavior, can we make it simpler to config? |
Fixes #1, Fixes #2I changed the order and added
browserslistConfigFile: false.