oxc-project / oxc

⚓ A collection of JavaScript tools written in Rust.
https://oxc.rs
MIT License
12.51k stars 458 forks source link

☂️ eslint-plugin-next #1929

Open camc314 opened 10 months ago

camc314 commented 10 months ago

[!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 @next/eslint-plugin-next.

There are 21(+ 0 deprecated) rules.

To get started, run the following command:

just new-nextjs-rule <RULE_NAME>

Then register the rule in crates/oxc_linter/src/rules.rs and also declare_all_lint_rules at the bottom.

Recommended rules

✨: 20, 🚫: 0 / total: 21 | Status | Name | Docs | | :----: | :--- | :--- | | ✨ | nextjs/google-font-display | https://nextjs.org/docs/messages/google-font-display | | ✨ | nextjs/google-font-preconnect | https://nextjs.org/docs/messages/google-font-preconnect | | ✨ | nextjs/inline-script-id | https://nextjs.org/docs/messages/inline-script-id | | ✨ | nextjs/next-script-for-ga | https://nextjs.org/docs/messages/next-script-for-ga | | ✨ | nextjs/no-assign-module-variable | https://nextjs.org/docs/messages/no-assign-module-variable | | ✨ | nextjs/no-async-client-component | https://nextjs.org/docs/messages/no-async-client-component | | ✨ | nextjs/no-before-interactive-script-outside-document | https://nextjs.org/docs/messages/no-before-interactive-script-outside-document | | ✨ | nextjs/no-css-tags | https://nextjs.org/docs/messages/no-css-tags | | ✨ | nextjs/no-document-import-in-page | https://nextjs.org/docs/messages/no-document-import-in-page | | ✨ | nextjs/no-duplicate-head | https://nextjs.org/docs/messages/no-duplicate-head | | ✨ | nextjs/no-head-element | https://nextjs.org/docs/messages/no-head-element | | ✨ | nextjs/no-head-import-in-document | https://nextjs.org/docs/messages/no-head-import-in-document | | | nextjs/no-html-link-for-pages | https://nextjs.org/docs/messages/no-html-link-for-pages | | ✨ | nextjs/no-img-element | https://nextjs.org/docs/messages/no-img-element | | ✨ | nextjs/no-page-custom-font | https://nextjs.org/docs/messages/no-page-custom-font | | ✨ | nextjs/no-script-component-in-head | https://nextjs.org/docs/messages/no-script-component-in-head | | ✨ | nextjs/no-styled-jsx-in-document | https://nextjs.org/docs/messages/no-styled-jsx-in-document | | ✨ | nextjs/no-sync-scripts | https://nextjs.org/docs/messages/no-sync-scripts | | ✨ | nextjs/no-title-in-document-head | https://nextjs.org/docs/messages/no-title-in-document-head | | ✨ | nextjs/no-typos | | | ✨ | nextjs/no-unwanted-polyfillio | https://nextjs.org/docs/messages/no-unwanted-polyfillio | ✨ = Implemented, 🚫 = No need to implement
Boshen commented 10 months ago

Feels like we need a --next-plugin?

camc314 commented 10 months ago

Feels like we need a --next-plugin?

yeah that makes sense

kaykdm commented 10 months ago

no-typos

@camc314 Can I work on this?

camc314 commented 10 months ago

no-typos

@camc314 Can I work on this?

yep, of course