Skip to content

feat(sveltekit): Create Sentry CLI config files#246

Merged
Lms24 merged 3 commits intolms/sveltekit-supportfrom
lms/sveltekit-cli-config
Apr 19, 2023
Merged

feat(sveltekit): Create Sentry CLI config files#246
Lms24 merged 3 commits intolms/sveltekit-supportfrom
lms/sveltekit-cli-config

Conversation

@Lms24
Copy link
Copy Markdown
Member

@Lms24 Lms24 commented Apr 17, 2023

This PR adds the creation of Sentry CLI config files to the SvelteKit SDK. Since the NextJS integration already does this, this PR extracts a couple of methods from the NextJS integration class to be reused for SvelteKit.

#skip-changelog

ref #244

@Lms24 Lms24 requested review from AbhiPrasad and lforst April 17, 2023 13:42
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 17, 2023

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

Generated by 🚫 dangerJS against 4c1eb25

@Lms24 Lms24 self-assigned this Apr 17, 2023
@Lms24 Lms24 force-pushed the lms/sveltekit-add-menu branch 4 times, most recently from c6632cd to e3e6dbc Compare April 19, 2023 12:05
Base automatically changed from lms/sveltekit-add-menu to lms/sveltekit-support April 19, 2023 12:07
@Lms24 Lms24 force-pushed the lms/sveltekit-cli-config branch 3 times, most recently from 11e1d31 to e1b5b8b Compare April 19, 2023 12:38
@Lms24 Lms24 force-pushed the lms/sveltekit-cli-config branch from e1b5b8b to 4c1eb25 Compare April 19, 2023 12:41
@Lms24 Lms24 merged commit 3a3a8fc into lms/sveltekit-support Apr 19, 2023
@Lms24 Lms24 deleted the lms/sveltekit-cli-config branch April 19, 2023 12:43
Lms24 added a commit that referenced this pull request Apr 19, 2023
adds the creation of Sentry CLI config files to the SvelteKit SDK setup
Lms24 added a commit that referenced this pull request Apr 21, 2023
adds the creation of Sentry CLI config files to the SvelteKit SDK setup
Lms24 added a commit that referenced this pull request May 3, 2023
adds the creation of Sentry CLI config files to the SvelteKit SDK setup
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.

2 participants