Skip to content

Conversation

sugitata
Copy link
Contributor

@sugitata sugitata commented Nov 13, 2023

Description

It's just a workaround like solution.
Add the defineNuxtComponent alias for Nuxt3 projects, similar to #1088.

About defineNuxtComponent

The difference between defineNuxtComponent and defineComponent is only asyncData and head options are added on defineNuxtComponent.
These options are not essential to check in vue-eslint-plugin, so it should be fine to treat it as defineComponent.

https://nuxt.com/docs/api/utils/define-nuxt-component

Copy link
Member

@FloEdelmann FloEdelmann left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@sugitata
Copy link
Contributor Author

@FloEdelmann
Thank you for your review!
I fixed them.

Copy link
Member

@FloEdelmann FloEdelmann left a comment

Choose a reason for hiding this comment

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

LGTM 🙂

@ota-meshi Should we consider this a breaking change and keep this for v10? (see #2166)

Copy link
Member

@ota-meshi ota-meshi left a comment

Choose a reason for hiding this comment

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

LGTM! Sorry for the late review.

@ota-meshi
Copy link
Member

Should we consider this a breaking change and keep this for v10?

Yeah. I think this should be included in v10.

@FloEdelmann FloEdelmann merged commit d27c7e6 into vuejs:v10 Nov 27, 2024
6 checks passed
@FloEdelmann FloEdelmann mentioned this pull request Nov 27, 2024
ota-meshi pushed a commit that referenced this pull request Nov 30, 2024
waynzh pushed a commit that referenced this pull request Feb 20, 2025
ota-meshi pushed a commit that referenced this pull request Mar 5, 2025
ota-meshi pushed a commit that referenced this pull request Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vue/no-unused-properties does not work with defineNuxtComponent() vue/order-in-components is not working with defineNuxtComponent()
3 participants