Skip to content
This repository was archived by the owner on Apr 6, 2023. It is now read-only.
/frameworkPublic archive

feat(nuxt, kit): auto import defineNuxtConfig - #7497

Merged
pi0 merged 4 commits into
mainfrom
feat/config-autoimport
Sep 14, 2022
Merged

feat(nuxt, kit): auto import defineNuxtConfig#7497
pi0 merged 4 commits into
mainfrom
feat/config-autoimport

Conversation

@pi0

@pi0pi0 commented Sep 14, 2022

Copy link
Copy Markdown
Member

🔗 Linked issue

#7485

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

A quick implementation to make defineNuxtConfig available when loading configuration. This way we can remove import { defineNuxtConfig } from 'nuxt/config'. If the feature proves to be stable, will probably move it to c12.

Type support: It is possible with a global reference defined in 'nuxt' types. Since we run nuxi prepare after installing dependencies, it should be immediately available.

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@pi0
pi0 requested a review from danielroeSeptember 14, 2022 10:52
@netlify

netlifyBot commented Sep 14, 2022

Copy link
Copy Markdown

Deploy Preview for nuxt3-docs canceled.

NameLink
🔨 Latest commit4ca6bcb
🔍 Latest deploy loghttps://app.netlify.com/sites/nuxt3-docs/deploys/6321b519bdafac00084d0adf

@pi0
pi0 merged commit ac0b3a6 into mainSep 14, 2022
@pi0
pi0 deleted the feat/config-autoimport branch September 14, 2022 11:10
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@pi0@danielroe