Skip to content

Conversation

@JPeer264
Copy link
Member

@JPeer264 JPeer264 commented Nov 20, 2025

This upgrades our internal eslint config to use v8. Theoretically this is considered a breaking change. Since this is an internal @sentry-internal/* package, we can release this in a non-breaking fashion

With v8 couple of changes came. Some of them are affecting us, which required file changes:

@github-actions
Copy link
Contributor

github-actions bot commented Nov 20, 2025

node-overhead report 🧳

Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.

Scenario Requests/s % of Baseline Prev. Requests/s Change %
GET Baseline 9,107 - 8,783 +4%
GET With Sentry 1,378 15% 1,354 +2%
GET With Sentry (error only) 6,119 67% 5,981 +2%
POST Baseline 1,209 - 1,200 +1%
POST With Sentry 512 42% 494 +4%
POST With Sentry (error only) 1,050 87% 1,036 +1%
MYSQL Baseline 3,342 - 3,258 +3%
MYSQL With Sentry 434 13% 464 -6%
MYSQL With Sentry (error only) 2,728 82% 2,673 +2%

View base workflow run

@JPeer264 JPeer264 requested review from Lms24 and s1gr1d November 20, 2025 15:44
@JPeer264 JPeer264 marked this pull request as ready for review November 20, 2025 15:45
@JPeer264 JPeer264 requested a review from a team as a code owner November 20, 2025 15:45
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.

2 participants