Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(nuxt): Delete no longer needed Nitro 'close' hook #15790

Merged
merged 1 commit into from
Mar 24, 2025

Conversation

s1gr1d
Copy link
Member

@s1gr1d s1gr1d commented Mar 24, 2025

Due to PR #15710 the Sentry server config is only processed inside the Nitro-part of Nuxt and the server config file is emitted through the Rollup plugin.

It is not needed anymore to copy-paste the file manually (this has been the case before as the file was added to the .nuxt folder - as it was processed by Nuxt, not Nitro-only).

This fixes the "no such file" error posted in this comment.

closes #13330

@s1gr1d s1gr1d requested review from lforst and Lms24 March 24, 2025 09:49
Copy link
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

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

Nice!

@s1gr1d s1gr1d merged commit 79ff8f2 into develop Mar 24, 2025
37 checks passed
@s1gr1d s1gr1d deleted the sig/nuxt-nitro-rollup branch March 24, 2025 09:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Nuxt module does not take into account "srcDir" for client instrumentation
3 participants