-
-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(nuxt): Add nuxt wizard #719
Conversation
|
* feat(nuxt): Add Nuxt menu option and install SDK * Mark sentry dot env file in cyan color
…r config files (#713) * feat(nuxt): Add sdk module to nuxt config and create client and server config files * Log snippets when user denies overwriting their configs
0ea8257
to
a3df6a0
Compare
* feat(nuxt): Add nuxt-3 e2e tests * Add nuxt-3 e2e tests * Add describe level * Add nuxt-4 e2e test app * Add nuxt 4 e2e test app * Remove leaking test helper
a3df6a0
to
011afbe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I used the wizard in an OSS nuxt project and it worked well! The steps seem fine to me, wording is great!
)} failed, please add the following settings:`, | ||
); | ||
// eslint-disable-next-line no-console | ||
console.log(`\n\n${getNuxtModuleFallbackTemplate(options)}\n\n`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
l (feel free to disregard if N/A): We have a diff-like helper function for displaying code snippets: makeCodeSnippet
in clack-utils.ts
. Might be useful here but I'll let you make the call if it's necessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I'll disregard for now to get this out, but will revisit.
Co-authored-by: Lukas Stracke <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andreiborza, this test app is added here as a submodule. I think that's why the Nuxt tests are broken on master.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, nice catch @onurtemizkan, I opened up a pr to fix it: #742
Closes: #705