Skip to content
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

docs(sveltekit): process.env.SENTRY_AUTH_TOKEN #12118

Merged
merged 1 commit into from
May 22, 2024

Conversation

pboling
Copy link
Contributor

@pboling pboling commented May 19, 2024

Remove quotes around process.env.SENTRY_AUTH_TOKEN

Before submitting a pull request, please take a look at our
Contributing guidelines and verify:

  • If you've added code that should be tested, please add tests.
  • Ensure your code lints and the test suite passes (yarn lint) & (yarn test).

Remove quotes around process.env.SENTRY_AUTH_TOKEN
@andreiborza andreiborza requested a review from Lms24 May 19, 2024 08:17
@pboling
Copy link
Contributor Author

pboling commented May 19, 2024

Worth noting that in spite of the ts in // vite.config.(js|ts) on line 208, many of the config options in this example do not work in Typescript. It's a separate issue, probably represented elsewhere in the issue tracker, but just wanted to point out that the documentation is still wrong, just slightly better.

@mydea mydea changed the base branch from master to develop May 21, 2024 08:04
@mydea mydea changed the title Fix documentation: process.env.SENTRY_AUTH_TOKEN docs(sveltekit): process.env.SENTRY_AUTH_TOKEN May 22, 2024
@mydea mydea merged commit 6cb60a3 into getsentry:develop May 22, 2024
27 of 28 checks passed
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