Skip to content

feat(nuxt): More granular error catching while modifying config - #833

Merged
s1gr1d merged 5 commits into
masterfrom
sig/nuxt-tags
Feb 28, 2025
Merged

s1gr1d merged 5 commits into
masterfrom
sig/nuxt-tags

Conversation

@s1gr1d

@s1gr1d s1gr1d commented Feb 28, 2025

Copy link
Copy Markdown
Member

Adds more try/catches around each step of modifying the Nuxt config. Each step fails by setting appropriate tags and displays an error message in the output.

@s1gr1d
s1gr1d requested a review from andreiborza February 28, 2025 12:29
@s1gr1d s1gr1d self-assigned this Feb 28, 2025
@s1gr1d
s1gr1d requested a review from smeubank February 28, 2025 12:29
@github-actions

github-actions Bot commented Feb 28, 2025 •

Copy link
Copy Markdown
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 143fa5e

@codecov

codecov Bot commented Feb 28, 2025 •

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.83%. Comparing base (e42459a) to head (143fa5e).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #833   +/-   ##
=======================================
  Coverage   50.83%   50.83%           
=======================================
  Files          53       53           
  Lines        3517     3517           
  Branches      825      825           
=======================================
  Hits         1788     1788           
  Misses       1601     1601           
  Partials      128      128           
Flag Coverage Δ
e2e-tests 80.85% <ø> (ø)
unit-tests 49.58% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment thread src/nuxt/sdk-setup.ts Outdated
Comment thread src/nuxt/sdk-setup.ts Outdated
@s1gr1d
s1gr1d requested a review from andreiborza February 28, 2025 14:37
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.

[nuxt] Error while setting up the Nuxt Module in nuxt config

2 participants