- Notifications
You must be signed in to change notification settings - Fork8
Add options to generate a Flat Config#25
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
1a9f098
to4edf18c
CompareUh oh!
There was an error while loading.Please reload this page.
This should now be ready for review! As for now, I'll go and test the various configs in |
Back with some results. In general, the common configs seem to work. The only parsing error I encountered was with scenario 8, which I haven't looked into. You can view the results in this Gist:https://gist.github.com/JoostKersjes/0e6495987a74df8ebf4527f35d924e27 |
juliandreas commentedApr 16, 2024
Just out of curiosity, why give the option of using the deprecated |
JoostKersjes commentedApr 16, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Made me think for a bit, thanks. I didn't want to be too disruptive with this MR so it could be more easily reviewed, so I didn't want to remove the old code. From my perspective, the adoption of flat configs has been slower than I expected. Most of the packages aren't compatible and only have instructions for using an Using packages wrapped in PS: It seems that only a small group of people would run |
Uh oh!
There was an error while loading.Please reload this page.
vuejs/create-vue#451
Things I have yet to do:
bin/create-eslint-config.js
changeseslint:recommended
oreslint:all
FlatCompat
from default + JS + no PrettiercreateAliasSetting
@eslint/js
&@eslint/eslintrc
when actually neededSome questions:
--ext
CLI flag gone, where should the file path match patterns come from? I thinkcreate-vue
can set them, but what about the manual create use case? Should both get the same defaultfiles: []
?--ignorePath
CLI flag is gone, should ignore patterns now also be part of this project? If so, should that be in this PR?