Skip to content

feat(lint-plugin-nuxt): add rules for Getting Started part of docs - #16

Open
Sanielia wants to merge 17 commits into
Zerya-Dev:masterfrom
Sanielia:feat/nuxt-rules-from-docs
Open

Sanielia wants to merge 17 commits into
Zerya-Dev:masterfrom
Sanielia:feat/nuxt-rules-from-docs

Conversation

@Sanielia

@Sanielia Sanielia commented Aug 5, 2026

Copy link
Copy Markdown
Member

Part of #7

To simplify reviewer work i will split rules derived from nuxt docs in multiple pull request packages added over time, so you have time to review what i did until now while I work on more rules, and you don't get 50k changes in one pr

@Sanielia
Sanielia requested a review from Norbiros August 5, 2026 16:29

@Norbiros Norbiros left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't review the code (yet), just the descriptions

Comment thread packages/lint-plugin-nuxt/src/rules/lazy-fetch-style/index.md Outdated
Comment thread packages/lint-plugin-nuxt/src/rules/no-invalid-status-text/index.md
@Norbiros Norbiros mentioned this pull request Aug 5, 2026
2 of 16 tasks
Comment thread packages/lint-plugin-nuxt/src/rules/lazy-fetch-style/index.md Outdated
@Sanielia
Sanielia force-pushed the feat/nuxt-rules-from-docs branch 2 times, most recently from ed6a7bb to 383a616 Compare August 11, 2026 19:23
@Sanielia
Sanielia force-pushed the feat/nuxt-rules-from-docs branch from 345c12b to c68106d Compare August 11, 2026 19:55
…`no-ignored-config-files` rule

Without this option monorepo with projects that require those files would have false positivies raised.
…definition in `no-ref-outside-setup` rule

Without this check any function named "setup" passed, which could generate false negatives
…nces in `no-async-data-after-mount` rule

Without this check passing composable directly as callback to function would allow execution after mount
@Sanielia
Sanielia force-pushed the feat/nuxt-rules-from-docs branch from c68106d to 6857a7c Compare August 11, 2026 20:09
@Sanielia
Sanielia requested a review from Norbiros August 12, 2026 18:03

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants