meta(changelog): Update changelog for 8.0.0-rc.0 #11916
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
[Gitflow] Merge master into develop
This reverts #11888, and ensures the feedback tests actually work on CDN. For this, we now ensure to serve this locally, so this will work also on release branches. It means you have to use `getLocalTestUrl` instead of `getLocalTestPath` to work. (side note: We can/should probably just remove `getLocalTestPath` overall 🤔 URL based is much more realistic and IMHO better. I'll do that in a follow up, maybe.
As some instrumentation may need higher versions, which we don't want to block there. This shuld be treated more like core etc. Note that deduping this does not really work nicely, sadly, because this is still experimental (`0.x`) so `^0.50.0` will not match `0.51.0` 😞
This PR bumps the hapi instrumentation to v0.38.0, which now supports hapi 21: https://github.com/open-telemetry/opentelemetry-js-contrib/releases/tag/instrumentation-hapi-v0.38.0 It also re-adds the hapi E2E test. Closes #11411
…rowser is small (#11893)
[See CHANGELOG](https://github.com/jsdom/jsdom/blob/main/Changelog.md) 21.1.2 is the latest version that still provides support for node v14
This prints a console.warn out if we detect that we are running in ESM mode.
mydea
reviewed
May 6, 2024
CHANGELOG.md
Outdated
Comment on lines
11
to
12
| We also recommend, reading the detailed | ||
| [migration guide](https://docs.sentry.io/platforms/javascript/migration/v7-to-v8/) in the docs. |
There was a problem hiding this comment.
Suggested change
| We also recommend, reading the detailed | |
| [migration guide](https://docs.sentry.io/platforms/javascript/migration/v7-to-v8/) in the docs. | |
| We recommend to read the detailed | |
| [migration guide](https://docs.sentry.io/platforms/javascript/migration/v7-to-v8/) in the docs. |
mydea
reviewed
May 6, 2024
CHANGELOG.md
Outdated
| Chrome 63 Edge 79 Safari/iOS Safari 12 Firefox 58 Opera 50 Samsung Internet 8.2 | ||
|
|
||
| For more details, please see the | ||
| [version support section in migration guide](https://github.com/getsentry/sentry-javascript/blob/8.0.0-beta.1/MIGRATION.md#1-version-support-changes). |
There was a problem hiding this comment.
Suggested change
| [version support section in migration guide](https://github.com/getsentry/sentry-javascript/blob/8.0.0-beta.1/MIGRATION.md#1-version-support-changes). | |
| [version support section in migration guide](https://github.com/getsentry/sentry-javascript/blob/develop/MIGRATION.md#1-version-support-changes). |
should this be the docs on develop? 🤔
mydea
reviewed
May 6, 2024
CHANGELOG.md
Outdated
|
|
||
| We now officially support Node 22. | ||
|
|
||
| - **fix(node): Fix nest.js error handler (#11874)** |
There was a problem hiding this comment.
I would move all these fixes down to "Other Changes" - important here is really the big, breaking stuff only!
There was a problem hiding this comment.
I added all important changes from the last beta releases plus Node 22 support and the one with Nest.js as the Nest.js fix is a breaking change, right?
mydea
reviewed
May 6, 2024
CHANGELOG.md
Outdated
| This change was made to integrate more nicely with the HTTP instrumentation added in earlier versions to avoid creating | ||
| unnecessary transactions. | ||
|
|
||
| - **feat(browser): Update `propagationContext` on `spanEnd` to keep trace consistent** |
There was a problem hiding this comment.
really all of these should probably be under "other changes" I believe, coming here you have no context of all the other hundreds of changes in the rc 😅
s1gr1d
force-pushed
the
prepare-release/v8.0.0-rc.0
branch
from
1a1e94c
to
fee4af9
Compare
mydea
reviewed
May 6, 2024
CHANGELOG.md
Outdated
Comment on lines
35
to
36
| When running Sentry in ESM mode, you will get a warning about that. Consider transpiling your application to CJS | ||
| (CommonJS) in your production build. |
There was a problem hiding this comment.
Suggested change
| When running Sentry in ESM mode, you will get a warning about that. Consider transpiling your application to CJS | |
| (CommonJS) in your production build. | |
| When running Sentry in ESM mode, we will now warn you that this is not supported as of now. We are working on ensuring support with ESM builds. |
maybe something like this? 🤔
mydea
reviewed
May 6, 2024
CHANGELOG.md
Outdated
Comment on lines
40
to
41
| - dev: Upgrade to `jsdom@21.1.2` (#11900) | ||
| - docs: fix typo in doc for `reactRouterV6BrowserTracingIntegration` (#11896) |
There was a problem hiding this comment.
Suggested change
| - dev: Upgrade to `jsdom@21.1.2` (#11900) | |
| - docs: fix typo in doc for `reactRouterV6BrowserTracingIntegration` (#11896) |
no need to highlight docs/dev changes in here :)
mydea
reviewed
May 6, 2024
CHANGELOG.md
Outdated
Comment on lines
15
to
26
| ### Version Support | ||
|
|
||
| The Sentry JavaScript SDK v8 now supports Node.js 14.8.0 or higher. This applies to @sentry/node and all of our | ||
| node-based server-side sdks (@sentry/nextjs, @sentry/remix, etc.). | ||
|
|
||
| The browser SDKs now require ES2018+ compatible browsers. New minimum browser versions: | ||
|
|
||
| Chrome 63 Edge 79 Safari/iOS Safari 12 Firefox 58 Opera 50 Samsung Internet 8.2 | ||
|
|
||
| For more details, please see the | ||
| [version support section in migration guide](https://github.com/getsentry/sentry-javascript/blob/develop/MIGRATION.md#1-version-support-changes). | ||
|
|
There was a problem hiding this comment.
Suggested change
| ### Version Support | |
| The Sentry JavaScript SDK v8 now supports Node.js 14.8.0 or higher. This applies to @sentry/node and all of our | |
| node-based server-side sdks (@sentry/nextjs, @sentry/remix, etc.). | |
| The browser SDKs now require ES2018+ compatible browsers. New minimum browser versions: | |
| Chrome 63 Edge 79 Safari/iOS Safari 12 Firefox 58 Opera 50 Samsung Internet 8.2 | |
| For more details, please see the | |
| [version support section in migration guide](https://github.com/getsentry/sentry-javascript/blob/develop/MIGRATION.md#1-version-support-changes). |
I think we can omit this as well here then, keep it a "normal"/"small" changelog!
mydea
approved these changes
May 6, 2024
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.