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
chore(schema): enable linting #22546
Conversation
packages/schema/src/types/nuxt.ts
Outdated
/** @deprecated use mode */ | ||
ssr?: boolean | ||
src: string | ||
mode?: 'all' | 'server' | 'client' | ||
/** | ||
* This allows more granular control over plugin order and should only be used by advanced users. | ||
* Lower numbers run first, and user plugins default to `0`. | ||
* | ||
* Default Nuxt priorities can be seen at [here](https://github.com/nuxt/nuxt/blob/9904849bc87c53dfbd3ea3528140a5684c63c8d8/packages/nuxt/src/core/plugins/plugin-metadata.ts#L15-L34). | ||
*/ | ||
order?: number |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/** @deprecated use mode */ | |
ssr?: boolean | |
src: string | |
mode?: 'all' | 'server' | 'client' | |
/** | |
* This allows more granular control over plugin order and should only be used by advanced users. | |
* Lower numbers run first, and user plugins default to `0`. | |
* | |
* Default Nuxt priorities can be seen at [here](https://github.com/nuxt/nuxt/blob/9904849bc87c53dfbd3ea3528140a5684c63c8d8/packages/nuxt/src/core/plugins/plugin-metadata.ts#L15-L34). | |
*/ | |
order?: number | |
/** @deprecated use mode */ | |
ssr?: boolean | |
src: string | |
mode?: 'all' | 'server' | 'client' | |
/** | |
* This allows more granular control over plugin order and should only be used by advanced users. | |
* Lower numbers run first, and user plugins default to `0`. | |
* | |
* Default Nuxt priorities can be seen at [here](https://github.com/nuxt/nuxt/blob/9904849bc87c53dfbd3ea3528140a5684c63c8d8/packages/nuxt/src/core/plugins/plugin-metadata.ts#L15-L34). | |
*/ | |
order?: number |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do you think this wasn't caught by eslint? π€
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It appears to have something to do with the folder name types
this file is in.
Renaming the folder to anything other than types
doesn't seem to make any difference. Odd!
π Linked issue
β Type of change
π Description
This PR enables linting for
packages/schema
and applies suggestions, as well as addresses errors resulting from this change.closes #22544
π Checklist