Skip to content

fix(nuxt): Add nuxt-module-builder for build output#13183

Merged
s1gr1d merged 6 commits intodevelopfrom
sig/add-module-builder-again
Aug 2, 2024
Merged

fix(nuxt): Add nuxt-module-builder for build output#13183
s1gr1d merged 6 commits intodevelopfrom
sig/add-module-builder-again

Conversation

@s1gr1d
Copy link
Copy Markdown
Member

@s1gr1d s1gr1d commented Aug 2, 2024

Reverting the change in this PR: #13138
For the time being (until this is debugged and fixed), this change is reverted and the Nuxt module will still be built with the nuxt-module-builder.

The changed build-setup led to an error in nuxt dev and the navigation in the E2E test apps did not work. The error message:

ERROR  [worker reload] [worker init] Package import specifier "#internal/nitro/virtual/app-config" is not defined in package /Users/sigridh/Documents/DEV/sentry-javascript-examples/node_modules/nitropack/package.json imported from /Users/sigridh/Documents/DEV/sentry-javascript-examples/node_modules/nitropack/dist/runtime/config.mjs

Similar errors were reported here:

@s1gr1d s1gr1d requested review from Lms24, andreiborza and lforst August 2, 2024 12:01
@s1gr1d s1gr1d merged commit 988d8e7 into develop Aug 2, 2024
@s1gr1d s1gr1d deleted the sig/add-module-builder-again branch August 2, 2024 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants