-
-
Notifications
You must be signed in to change notification settings - Fork 49
feat: add config "flat/all" #401
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
Conversation
6acee02
to
1d59eae
Compare
164bb72
to
cb4be05
Compare
cb4be05
to
2e13d59
Compare
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.
I don't think this is a good idea #400 (comment)
If we document this the same way as ESLint did in eslint/eslint#19318 then I'm okay with this 👍 |
This comment was marked as resolved.
This comment was marked as resolved.
If we have clear documentation, I'm fine with merging this as well |
4fbfb26
to
190d265
Compare
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.
Generally 👍 to this documentation is clear and the recommended rules detection is super nice.
open questions: 1. need to add an equivalent eslintrc config? fixes #400
40cb730
to
41bbbed
Compare
open questions:
2. need to add language options?fixes #400