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

meta(changelog): Update changelog for 8.19.0 #12976

Merged
merged 23 commits into from
Jul 18, 2024
Merged

Conversation

andreiborza
Copy link
Member

No description provided.

lforst and others added 22 commits July 16, 2024 14:21
[Gitflow] Merge master into develop
…acks (#12930)

Add a second parameter (`request`) to the `httpIntegration`'s
`ignoreIncomingRequests` and `ignoreOutgoingRequests` callbacks. This
allows users to not only filter on the URL as previously but on the
entire request objects.
This adds docs about how to resolve gitflow merge conflicts.

---------

Co-authored-by: Andrei <168741329+andreiborza@users.noreply.github.com>
I noticed these while working on the cloudflare sdk.

- update browser path to use new replay packages

We were previously referencing `packages/replay`, which does not exist
anymore. This updates us to use the correct paths.

- add vercel-edge to nextjs

The nextjs sdk depends on vercel-edge, so we should be running tests for
it when that package changes.

- remove browser from deno

The deno sdk no longer relies on the browser sdk, this updates the paths
accordingly

- add bun section

We can be smarter about CI time and only run bun tests when relevant
package paths change.
Before: `Time: 2.621 s`

After: `Duration 638ms (transform 343ms, setup 0ms, collect 1.61s, tests
22ms, environment 0ms, prepare 278ms)`

ref #11084

Also removes `edge-runtime/jest-environment` which we weren't using
anyway, removes a lot of unnecessary stuff in our `yarn.lock` (-151
lines!)
Before: `Time: 0.303 s`

After: `Duration 271ms (transform 28ms, setup 0ms, collect 26ms, tests
3ms, environment 0ms, prepare 68ms)`
Before: `Time:        2.026 s`

After: `Duration 960ms (transform 291ms, setup 0ms, collect 446ms, tests
5ms, environment 283ms, prepare 65ms)`

Also:

- removes unused `@babel/core` dev dep
- removes unused eslintrc logic
SentryCron was missing from the nest readme
…st applications (#12948)

Refactor of the existing nestjs test applications. Before we had one
sample application testing everything nest-related. This PR splits them
up into three applications to make it more readable and easier to
understand what is being tested. It also allows for iterating a bit
quicker in local development.

No new functionality was added. Will add more tests in a follow-up.

The three new services are:
- nestjs-basic: Simple nestjs application with no submodules and tests
for basic functionality of the SDK like error monitoring and span
instrumentation.
- nestjs-with-submodules: NestJS application that is bit more complex
including a submodule (and potentially multiple in the future) to have a
more realistic setup for more advanced testing.
- nestjs-distributed-tracing: Includes tests for trace propagation with
multiple services.
…MContentLoaded event is triggered (#12973)

Fixing a missing paranthesis to call `appendToDom` on `DOMContentLoaded`
event.

Fixes #12970
This PR adds the external contributor to the CHANGELOG.md file, so that
they are credited for their contribution. See #12973

---------

Co-authored-by: andreiborza <168741329+andreiborza@users.noreply.github.com>
Co-authored-by: Andrei Borza <andrei.borza@sentry.io>
…destroyed (#12786)

Add cleanup logic to handle the removal of afterSendEvent, 
which is set up within the Angular error handler.
This fixes memory leaks that occur when the event is still 
being handled after the root view is removed
---------

Co-authored-by: Lukas Stracke <lukas.stracke@sentry.io>
This PR adds the external contributor to the CHANGELOG.md file, so that
they are credited for their contribution. See #12786

---------

Co-authored-by: Lms24 <8420481+Lms24@users.noreply.github.com>
Co-authored-by: Andrei Borza <andrei.borza@sentry.io>
@andreiborza andreiborza requested a review from Lms24 July 18, 2024 12:47
@andreiborza andreiborza changed the base branch from develop to master July 18, 2024 12:47
@andreiborza andreiborza requested review from a team as code owners July 18, 2024 12:47
CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Contributor

size-limit report 📦

Path Size
@sentry/browser 22.3 KB (added)
@sentry/browser (incl. Tracing) 33.72 KB (added)
@sentry/browser (incl. Tracing, Replay) 69.81 KB (added)
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 63.11 KB (added)
@sentry/browser (incl. Tracing, Replay with Canvas) 74.2 KB (added)
@sentry/browser (incl. Tracing, Replay, Feedback) 86.52 KB (added)
@sentry/browser (incl. Tracing, Replay, Feedback, metrics) 88.39 KB (added)
@sentry/browser (incl. metrics) 26.62 KB (added)
@sentry/browser (incl. Feedback) 38.98 KB (added)
@sentry/browser (incl. sendFeedback) 26.93 KB (added)
@sentry/browser (incl. FeedbackAsync) 31.54 KB (added)
@sentry/react 25.06 KB (added)
@sentry/react (incl. Tracing) 36.79 KB (added)
@sentry/vue 26.44 KB (added)
@sentry/vue (incl. Tracing) 35.6 KB (added)
@sentry/svelte 22.44 KB (added)
CDN Bundle 23.52 KB (added)
CDN Bundle (incl. Tracing) 35.5 KB (added)
CDN Bundle (incl. Tracing, Replay) 69.91 KB (added)
CDN Bundle (incl. Tracing, Replay, Feedback) 75.17 KB (added)
CDN Bundle - uncompressed 69 KB (added)
CDN Bundle (incl. Tracing) - uncompressed 105.1 KB (added)
CDN Bundle (incl. Tracing, Replay) - uncompressed 216.88 KB (added)
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 229.6 KB (added)
@sentry/nextjs (client) 36.64 KB (added)
@sentry/sveltekit (client) 34.37 KB (added)
@sentry/node 111.28 KB (added)
@sentry/node - without tracing 88.73 KB (added)
@sentry/aws-serverless 97.87 KB (added)

@andreiborza andreiborza merged commit ca99121 into master Jul 18, 2024
121 checks passed
@andreiborza andreiborza deleted the prepare-release/8.19.0 branch July 18, 2024 13:27
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.

9 participants