Check eslint-plugin-unicorn rules to enable/configure/disable #2439
Description
Followup on #2418
Currently all the following rules are disabled because they cause issues in the code:
Undecided
| Rule | Count | Status/PR |
|---|---|---|
unicorn/prevent-abbreviations |
232 | 🤔 - #3320 |
Last updated: 2023-10-07
Rejected
| Rule | Count | Status/PR |
|---|---|---|
unicorn/import-style |
8 | 🔴 - #2901 - doesn't do anything for us |
unicorn/no-nested-ternary |
N/A | 🔴 - incompatible with prettier |
unicorn/no-null |
N/A | 🔴 - incompatible with TypeScript |
unicorn/no-object-as-default-parameter |
6 | 🔴 - Unicorn-Issue |
unicorn/no-zero-fractions |
17 | 🔴 - #2453 |
unicorn/numeric-separators-style |
131 | 🔴 - #2815 |
unicorn/number-literal-case |
N/A | 🔴 - incompatible with prettier |
unicorn/prefer-string-slice |
75 | 🔴 - string.subtring is sometimes easier to understand - #3247 |
unicorn/prefer-ternary |
8 | 🔴 - #2464 |
Accepted
Please leave a comment if you wish to tackle any of these.
Legend:
- 🤔 : This needs a decision whether we want to enable this.
- 🔴 : We will not enable this rule.
- 🟡 : We probably want to enable this rule, but maybe with additional configuration.
- 🟢 : We want to enable this rule. Contributions welcome.
#1234: Reference to the related PR.
The above list can be updated using the following command:
pnpm run lint | grep 'unicorn/' | sed 's/.*unicorn/unicorn/' | sort | uniq -c | sed -E 's/ *(.*) unicorn\/(.*)/\| [`unicorn\/\2`](https:\/\/github.com\/sindresorhus\/eslint-plugin-unicorn\/blob\/main\/docs\/rules\/\2.md) \| \1 \| 🤔 \|/'