Skip to content

meta: Add Changelog entry for 8.7.0#12280

Merged
Lms24 merged 11 commits intomasterfrom
prepare-release/8.7.0
May 29, 2024
Merged

meta: Add Changelog entry for 8.7.0#12280
Lms24 merged 11 commits intomasterfrom
prepare-release/8.7.0

Conversation

@Lms24
Copy link
Copy Markdown
Member

@Lms24 Lms24 commented May 29, 2024

The 8.6.0 release failed halfway through - this release should hopefully fix it.

andreiborza and others added 10 commits May 29, 2024 11:15
This adds a new E2E test `react-send-to-sentry` that is run optionally.

For now this is the same as the old `standard-frontend-react` test -
eventually we can/should update that test (and others) to stop sending
to sentry.

This test will run in a separate group that we do not block merging on
when it fails.

For now, there is one browser and one node test that checks that they
send events successfully to Sentry - IMHO that should cover stuff
decently for now. I also made the source maps debug ID test optional, as
that inherently sends to Sentry.

We can in follow ups get rid of all the event sending stuff from the
remaining E2E tests.

Part of #11910
No need for it anymore, and actually it is much simpler using fetch
because that does not throw when encountering a 404 error!
Instead of just testing to send to Sentry, this now tests the actual
payloads being sent.

This kind of builds on top of
#12259, where we
specifically test the event sending now.
I noticed that locally the node-integration-tests that check versions
fail, because in the SDK we use 18.18.0, we only check for "If
node.major >= 18", and then get an incorrect test because actually in
code we check the minor too.

Instead of adjust this, IMHO it's better to update this. This also never
failed on CI because there we do use latest 18 for the matrix tests...
This was not really needed, so we may as well remove it. We still use it
in package scripts, which is fine I'd say.

Note that this still does not "fix" the `yarn add` issue, but one step
at a time...
Co-authored-by: Luca Forstner <luca.forstner@sentry.io>
…extConfig.productionBrowserSourceMaps: true` is set (#12278)
@Lms24 Lms24 requested a review from lforst May 29, 2024 15:34
@Lms24 Lms24 changed the base branch from develop to master May 29, 2024 15:35
@github-actions
Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size
@sentry/browser 21.74 KB (added)
@sentry/browser (incl. Tracing) 32.73 KB (added)
@sentry/browser (incl. Tracing, Replay) 68.3 KB (added)
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 61.62 KB (added)
@sentry/browser (incl. Tracing, Replay with Canvas) 72.36 KB (added)
@sentry/browser (incl. Tracing, Replay, Feedback) 84.44 KB (added)
@sentry/browser (incl. Tracing, Replay, Feedback, metrics) 86.29 KB (added)
@sentry/browser (incl. metrics) 25.91 KB (added)
@sentry/browser (incl. Feedback) 37.86 KB (added)
@sentry/browser (incl. sendFeedback) 26.32 KB (added)
@sentry/browser (incl. FeedbackAsync) 30.83 KB (added)
@sentry/react 24.51 KB (added)
@sentry/react (incl. Tracing) 35.78 KB (added)
@sentry/vue 25.72 KB (added)
@sentry/vue (incl. Tracing) 34.57 KB (added)
@sentry/svelte 21.87 KB (added)
CDN Bundle 23.11 KB (added)
CDN Bundle (incl. Tracing) 34.46 KB (added)
CDN Bundle (incl. Tracing, Replay) 68.39 KB (added)
CDN Bundle (incl. Tracing, Replay, Feedback) 73.53 KB (added)
CDN Bundle - uncompressed 68 KB (added)
CDN Bundle (incl. Tracing) - uncompressed 102.1 KB (added)
CDN Bundle (incl. Tracing, Replay) - uncompressed 212 KB (added)
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 224.37 KB (added)
@sentry/nextjs (client) 35.12 KB (added)
@sentry/sveltekit (client) 33.36 KB (added)
@sentry/node 115.24 KB (added)
@sentry/aws-serverless 103.73 KB (added)

@Lms24 Lms24 enabled auto-merge May 29, 2024 15:47
@Lms24 Lms24 merged commit d4aaa3a into master May 29, 2024
@Lms24 Lms24 deleted the prepare-release/8.7.0 branch May 29, 2024 15:52
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.

6 participants