Skip to content

feat(schema): default to bundler module resolution#24837

Merged
danielroe merged 2 commits intomainfrom
fix/module-resolution
Jan 18, 2024
Merged

feat(schema): default to bundler module resolution#24837
danielroe merged 2 commits intomainfrom
fix/module-resolution

Conversation

@danielroe
Copy link
Copy Markdown
Member

@danielroe danielroe commented Dec 20, 2023

🔗 Linked issue

#22821
22ce98d

❓ Type of change

  • 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
  • 🐞 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)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

(This is a remake of #22821.)

This opts us in to TypeScript bundler resolution which should more closely resemble the actual way that we resolve subpath imports for modules in Nuxt projects.

'Bundler' module resolution is recommended by Vue and by Vite, but unfortunately there are still many packages that do not have the correct entries in their package.json.

We'll merge for v3.10 as well as helping ecosystem update in the mean time.

📝 Checklist

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

@bolt-new-by-stackblitz
Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@danielroe danielroe added this to the 3.10 milestone Dec 20, 2023
@danielroe danielroe changed the title feat(schema): default to bundler module resolution (#22821) feat(schema): default to bundler module resolution Dec 20, 2023
@danielroe danielroe marked this pull request as draft December 20, 2023 14:21
This was referenced Dec 21, 2023
@danielroe danielroe force-pushed the fix/module-resolution branch from 9d508b0 to bdf8dcc Compare January 2, 2024 12:17
@danielroe
Copy link
Copy Markdown
Member Author

/ecosystem-ci run

@nuxt nuxt deleted a comment from nuxt-ecosystem-ci bot Jan 2, 2024
@danielroe danielroe marked this pull request as ready for review January 17, 2024 10:14
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.

1 participant