Skip to content

Commit 8c74e93

Browse files
committedSep 6, 2021
Improve emojification of rule changes
1 parent 2a666d4 commit 8c74e93

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed
 

‎README.md

+9-3
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,18 @@ Then add a `.eslintrc` with the following:
3535

3636
## How does this differ from pure [standard](https://standardjs.com/)?
3737

38+
* :stop_sign: = changed to `error` level
39+
* :warning: = changed to `warn` level
40+
* :mute: = deactivated
41+
* :wrench: = changed config
42+
* :grimacing: = will not pass vanilla [standard](https://standardjs.com/) linting
43+
3844
### :wrench: Changed [standard](https://standardjs.com/) rules
3945

40-
* :wrench::warning: [`comma-dangle`](https://eslint.org/docs/rules/comma-dangle)*changed* – ignore it in everything but functions + is it set to `warn` rather than `error`. Reason: I'm moving towards preferring dangling commas and thus want my shared config to not prohibit them while a project-by-project override can actually require them (Incompatible with standard :warning:)
41-
* :wrench: [`no-multi-spaces`](https://eslint.org/docs/rules/no-multi-spaces)*changed* – sets `ignoreEOLComments` to `true`, can be useful for more readable comments across multiple lines and I see no real downsides to it (Incompatible with standard :warning:)
46+
* :wrench::warning::grimacing: [`comma-dangle`](https://eslint.org/docs/rules/comma-dangle)*changed* – ignore it in everything but functions + is it set to `warn` rather than `error`. Reason: I'm moving towards preferring dangling commas and thus want my shared config to not prohibit them while a project-by-project override can actually require them (Incompatible with standard)
47+
* :wrench::grimacing: [`no-multi-spaces`](https://eslint.org/docs/rules/no-multi-spaces)*changed* – sets `ignoreEOLComments` to `true`, can be useful for more readable comments across multiple lines and I see no real downsides to it (Incompatible with standard)
4248
* :wrench: [`no-unused-vars`](https://eslint.org/docs/rules/no-unused-vars)*changed* – sets `"args": "all", "argsIgnorePattern": "^_",` because I personally don't feel limited by [Express error handlers](https://github.com/standard/standard/issues/419#issuecomment-718186130) + wants to stay in sync with TypeScript `noUnusedParameters`
43-
* :wrench: [`semi`](https://eslint.org/docs/rules/semi) and [`no-extra-semi`](https://eslint.org/docs/rules/no-extra-semi)*changed* – adopts the semicolons setup from [semistandard](https://github.com/standard/semistandard) (Incompatible with plain standard :warning:, compatible with semistandard)
49+
* :wrench::grimacing: [`semi`](https://eslint.org/docs/rules/semi) and [`no-extra-semi`](https://eslint.org/docs/rules/no-extra-semi)*changed* – adopts the semicolons setup from [semistandard](https://github.com/standard/semistandard) (Incompatible with plain standard, compatible with semistandard)
4450
* :wrench::warning: [`node/no-deprecated-api`](https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-deprecated-api.md)*changed* – changed to `warn` instead of `error` as often it's not an urgent thing to fix
4551

4652
### :heavy_plus_sign: Added ESLint core rules

0 commit comments

Comments
 (0)
Please sign in to comment.