Skip to content

Conversation

@Lms24
Copy link
Member

@Lms24 Lms24 commented May 8, 2024

Adds the changelog entry for 8.0.0 (stable). Still in draft - please feel free to add your thoughts what else should (not) be in here. Most of this is based on the 8.0.0-beta.0 entry but I swapped orders, rewrote the ESM/SDK init section and linked to docs in favour of MIGRATION.md where applicable.

General idea: Mention most important changes on a high-level; link to details and advanced migration instructions.

We can either merge this and make an additional prepare-release/ branch or simply close this PR and throw the entry onto the branch directly.

Copy link
Member

@andreiborza andreiborza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ESM/--require part LGTM

Copy link
Member

@HazAT HazAT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can extend this section and talk about the require flag more
https://docs.sentry.io/platforms/javascript/guides/node/migration/v7-to-v8/#updated-sdk-initialization

I know not related to this changelog but we should update the docs.

@Lms24
Copy link
Member Author

Lms24 commented May 8, 2024

@HazAT yup good point, already pointed this out in getsentry/sentry-docs#9932 (review)

@Lms24 Lms24 force-pushed the lms/meta-changelog-8.0.0 branch from 76b3ae2 to 56c93cd Compare May 13, 2024 14:19
@Lms24
Copy link
Member Author

Lms24 commented May 13, 2024

Superseded by #12002 to trigger release action w/ branch name

@Lms24 Lms24 closed this May 13, 2024
@Lms24 Lms24 deleted the lms/meta-changelog-8.0.0 branch December 3, 2024 09:31
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.

5 participants