Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

More obvious deprecation of Nuxt 2 settings in nuxt.config.js #15006

Closed
3 of 4 tasks
manniL opened this issue Sep 22, 2022 · 3 comments 路 Fixed by nuxt/framework#8487
Closed
3 of 4 tasks

More obvious deprecation of Nuxt 2 settings in nuxt.config.js #15006

manniL opened this issue Sep 22, 2022 · 3 comments 路 Fixed by nuxt/framework#8487

Comments

@manniL
Copy link
Member

manniL commented Sep 22, 2022

Describe the feature

Hey 馃憢馃徎

right now, it seems difficult for several people to realize which config options from nuxt.config.js/nuxt.config.ts are actually used in Nuxt 3 (e.g. target). Right now, the only hint is the @version tag in the JSDocs which isn't very prominent though.

image

It'd be nice to have a better alternative (e.g. @deprecated?) so people can be more aware.

Additional information

  • Would you be willing to help implement this feature?
  • Could this feature be implemented as a module?

Final checks

@danielroe
Copy link
Member

This is absolutely planned, though I couldn't find an issue for it.

@manniL
Copy link
Member Author

manniL commented Sep 22, 2022

though I couldn't find an issue for it.

I think this is usually a "side effect" in other Nuxt3 issues, e.g.:

But given the amount of people writing target in their Nuxt3 config, I assume that there is a good potential for improving DX 馃構

@pi0
Copy link
Member

pi0 commented Sep 22, 2022

Thanks for making issue @manniL. Indeed we need to move Nuxt 2 types to bridge and mark them as never for 3+ schema to avoid accidentally using them. I had planned to work on this soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants