-
-
Notifications
You must be signed in to change notification settings - Fork 477
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
☂️ typescript-eslint #503
Comments
Yes! Let's port the recommended list that don't require type information. As a bonus, we should copy over the test suite for extension rules. @nulladdict Keen to join the fun? |
Not sure if I have enough rust experience and enough domain knowledge, so I wouldn't count on me :) |
No worries |
@nulladdict @Boshen the |
implement @typescript-eslint/prefer-as-const Related issue: #503.
It's moved to See https://typescript-eslint.io/rules/prefer-namespace-keyword |
This comment was marked as resolved.
This comment was marked as resolved.
@Thiry1 Thank you, I updated the main list. |
…726) implement [@typescript-eslint/no-duplicate-enum-values](https://typescript-eslint.io/rules/no-duplicate-enum-values) Related issue: #503.
related to #503, marked as nursery(though all tests pass) since we use rust `regex` crate to parse the regex, may have compatibility problems with ECMA regex.
Part of #503 -> `@typescript-eslint/ban-types`
) implement @typescript-eslint/triple-slash-reference Related issue: #503 original - doc: https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/triple-slash-reference.md - code: https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/rules/triple-slash-reference.ts
Will type checking rules be considered for implementation at any point in time? |
Close in favor of #2180 so the creator this issue don't get spammed. |
…c-project#1903) implement @typescript-eslint/triple-slash-reference Related issue: oxc-project#503 original - doc: https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/triple-slash-reference.md - code: https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/rules/triple-slash-reference.ts
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 tracking issue for
@typescript-eslint/eslint-plugin
.Rules
Getting started
Then register the rule in
crates/oxc_linter/src/rules.rs
and alsodeclare_all_lint_rules
at the bottom.The text was updated successfully, but these errors were encountered: