-
-
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
base: master
Are you sure you want to change the base?
Conversation
@@ -17,7 +17,7 @@ module.exports = { | |||
type: "suggestion", | |||
docs: { | |||
description: "disallow the use of `process.exit()`", | |||
recommended: false, | |||
recommended: true, |
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.
seems it's a bug: it was false
, but was enabled in the recommended config
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 👍 |
I too am in agreement if we make the documentation clear for this. |
If we have clear documentation, I'm fine with merging this as well |
open questions: 1. need to add an equivalent eslintrc config? fixes #400
2e13d59
to
4fbfb26
Compare
4fbfb26
to
190d265
Compare
open questions:
2. need to add language options?fixes #400