Skip to content

Conversation

@JPeer264
Copy link
Member

It seems that the size limit was not enforced and a PR could be merged without the job being green, see:

Screenshot 2025-10-23 at 10 12 20

@JPeer264 JPeer264 self-assigned this Oct 23, 2025
@github-actions
Copy link
Contributor

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,520 - 9,475 +0%
GET With Sentry 1,428 15% 1,349 +6%
GET With Sentry (error only) 6,241 66% 6,020 +4%
POST Baseline 1,215 - 1,189 +2%
POST With Sentry 531 44% 513 +4%
POST With Sentry (error only) 1,072 88% 1,072 -
MYSQL Baseline 3,340 - 3,320 +1%
MYSQL With Sentry 461 14% 431 +7%
MYSQL With Sentry (error only) 2,708 81% 2,688 +1%

View base workflow run

@JPeer264 JPeer264 merged commit 1513161 into develop Oct 23, 2025
107 of 109 checks passed
@JPeer264 JPeer264 deleted the jp/require-size-check branch October 23, 2025 14:18
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