-
-
Notifications
You must be signed in to change notification settings - Fork 942
☂️ eslint-plugin-n #493
Copy link
Copy link
Open
Labels
A-linterArea - LinterArea - LinterE-Help WantedExperience level - For the experienced collaboratorsExperience level - For the experienced collaborators
Description
Warning
This comment is maintained by CI. Do not edit this comment directly.
To update comment template, see https://github.com/oxc-project/oxc/tree/main/tasks/lint_rules
This is the tracking issue for eslint-plugin-n.
There are 41(+ 2 deprecated) rules.
- 13/15 recommended rules are remaining as TODO
- 17/26 not recommended rules are remaining as TODO
To get started, run the following command:
just new-n-rule <RULE_NAME>Then implement the rule and get all the tests passing.
Recommended rules
✅: 1, 🚫: 1, ⏳: 0 / total: 15
| Status | Name | Rationale |
|---|---|---|
| n/no-unsupported-features/es-builtins | ||
| n/no-unsupported-features/es-syntax | ||
| n/hashbang | ||
| n/no-deprecated-api | ||
| ✅ | n/no-exports-assign | |
| n/no-extraneous-import | ||
| n/no-extraneous-require | ||
| n/no-missing-import | ||
| n/no-missing-require | ||
| 🚫 | n/no-process-exit | No need to implement, already implemented by unicorn/no-process-exit. |
| n/no-unpublished-bin | ||
| n/no-unpublished-import | ||
| n/no-unpublished-require | ||
| n/no-unsupported-features/node-builtins | ||
| n/process-exit-as-throw |
✅ = Implemented, 🚫 = Not intending to implement, ⏳ = Fix pending
Not recommended rules
✅: 5, 🚫: 4, ⏳: 0 / total: 26
| Status | Name | Rationale |
|---|---|---|
| n/prefer-global/buffer | ||
| n/callback-return | ||
| n/prefer-global/console | ||
| n/prefer-global/crypto | ||
| n/prefer-promises/dns | ||
| n/exports-style | ||
| 🚫 | n/file-extension-in-import | No need to implement, already implemented by import/extensions. |
| n/prefer-promises/fs | ||
| ✅ | n/global-require | |
| ✅ | n/handle-callback-err | |
| n/no-callback-literal | ||
| n/no-mixed-requires | ||
| ✅ | n/no-new-require | |
| ✅ | n/no-path-concat | |
| ✅ | n/no-process-env | |
| 🚫 | n/no-restricted-import | No need to implement, already implemented by no-restricted-imports rule. |
| 🚫 | n/no-restricted-require | No need to implement, already implemented by no-restricted-imports rule. |
| n/no-sync | ||
| n/no-top-level-await | ||
| 🚫 | n/prefer-node-protocol | No need to implement, already implemented by unicorn/prefer-node-protocol. |
| n/prefer-global/process | ||
| n/prefer-global/text-decoder | ||
| n/prefer-global/text-encoder | ||
| n/prefer-global/timers | ||
| n/prefer-global/url | ||
| n/prefer-global/url-search-params |
✅ = Implemented, 🚫 = Not intending to implement, ⏳ = Fix pending
Deprecated rules
✅: 0, 🚫: 2, ⏳: 0 / total: 2
| Status | Name | Rationale |
|---|---|---|
| 🚫 | n/no-hide-core-modules | This rule is deprecated in eslint-plugin-n for being inherently incorrect, no need for us to implement it. |
| 🚫 | n/shebang | Replaced by node/hashbang. |
✅ = Implemented, 🚫 = Not intending to implement, ⏳ = Fix pending
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-linterArea - LinterArea - LinterE-Help WantedExperience level - For the experienced collaboratorsExperience level - For the experienced collaborators
Type
Fields
Give feedbackPriority
None yet