-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
meta(changelog): Update changelog for 8.12.0 #12637
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Adds the correct link to the auto-release workflow.
…et (#12562) ref getsentry/sentry-release-registry#161 and #12442 Adds these packages to the craft config so they can be updated in the release registry.
Marks `isRemixV2` as optional in the exposed types of `captureRemixServerException`. This is a follow-up on #12497
This adds docs about how to triage effectively. It is rather specific to this repository/team, but parts of it may also be relevant for outside contributors. See https://github.com/getsentry/sentry-javascript/blob/fn/docs-triaging/docs/triaging.md --------- Co-authored-by: Abhijeet Prasad <aprasad@sentry.io>
11 and 12 are soon to be [deprecated](https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners) so we need to upgrade them
…ot integration (#12535) I believe the error we're seeing in #12534 is from having preact bundled twice: once into the Modal integration, and then again into the Screenshot one. This PR updates the Screenshot code so that it requires preact to be injected into it, which should reduce the bundle size of that integration, but also solve the bug because there will only be one instance of preact and it's state within the sdk. Fixes #12534
…12617)⚠️ This PR introduces **breaking changes** to `@sentry/solid`⚠️ Previously, we had to pass `useBeforeLeave` and `useLocation` to `solidRouterBrowserTracingIntegration`. This has now been removed in favor of importing the router hooks directly within the sdk as needed. Import `solidRouterBrowserTracingIntegration` from `@sentry/solid/solidrouter` and add it to `Sentry.init`: ```js import * as Sentry from '@sentry/solid'; import { solidRouterBrowserTracingIntegration, withSentryRouterRouting } from '@sentry/solid/solidrouter'; import { Router } from '@solidjs/router'; Sentry.init({ dsn: '__PUBLIC_DSN__', integrations: [solidRouterBrowserTracingIntegration()], tracesSampleRate: 1.0, // Capture 100% of the transactions }); const SentryRouter = withSentryRouterRouting(Router); ``` As a result, the SDK has an optional peer dependency on `@solidjs/router` `v0.13.4+` when using `solidRouterBrowserTracingIntegration`. **Note to maintainers** This package outputs types at the `build` root instead of `build/types` to better support projects that don't use `moduleResolution: "bundler"`. --------- Co-authored-by: Francesco Novy <francesco.novy@sentry.io> Co-authored-by: Luca Forstner <luca.forstner@sentry.io>
andreiborza
reviewed
Jun 25, 2024
size-limit report 📦
|
Don't forget to update the PR title, the auto-title generation does not work for changelog PRs like this ;) |
chargome
changed the title
Prepare release/8.12.0
meta(changelog): Update changelog for 8.12.0
Jun 25, 2024
andreiborza
approved these changes
Jun 25, 2024
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.
🚀
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.