FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

test(e2e): Move react-router-8-cloudflare MySQL tests into react-router-8-framework by JPeer264 · Pull Request #24599 · getsentry/sentry-javascript · GitHub

  1. getsentry
  2. sentry-javascript

Repository navigation

test(e2e): Move react-router-8-cloudflare MySQL tests into react-router-8-framework - #24599

Merged
JPeer264 merged 1 commit into
developfrom
jp/react-router-8-cloudflare-into-framework
Oct 5, 2026
ConversationCommits (1)ChecksFiles changed
Merged

test(e2e): Move react-router-8-cloudflare MySQL tests into react-router-8-framework#24599
JPeer264 merged 1 commit into
developfrom
jp/react-router-8-cloudflare-into-framework

Conversation

JPeer264 commented Sep 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

part of #24836

With the Cloudflare variant of react-router-8-framework, the separate react-router-8-cloudflare app only added its MySQL test. This moves the MySQL route and both tests into react-router-8-framework and removes the old app, so one app covers React Router on Node, Bun, Deno and Cloudflare.

The MySQL tests now run on every variant, not only on Cloudflare. The Worker is wrapped by sentryCloudflareVitePlugin instead of a manual withSentry call; other Cloudflare apps still cover manual withSentry.

🤖 Generated with Claude Code

JPeer264 added this pull request to stack #24600 September 22, 2026 16:42

github-actions Bot commented Sep 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

size-limit report 📦

⚠️ Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

Path Size % Change Change
@sentry/browser 29.52 kB - -
@sentry/browser - with treeshaking flags 27.68 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 27.57 kB - -
@sentry/browser (incl. Tracing) 51.45 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 51.46 kB - -
@sentry/browser (incl. Tracing, Profiling) 54.46 kB - -
@sentry/browser (incl. Tracing, Replay) 91.04 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 80.03 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 95.74 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 108.71 kB - -
@sentry/browser (incl. Feedback) 47.04 kB - -
@sentry/browser (incl. sendFeedback) 34.57 kB - -
@sentry/browser (incl. FeedbackAsync) 39.68 kB - -
@sentry/browser (incl. Metrics) 30.54 kB - -
@sentry/browser (incl. Logs) 30.82 kB - -
@sentry/browser (incl. Metrics & Logs) 31.48 kB - -
@sentry/react 31.36 kB - -
@sentry/react (incl. Tracing) 53.8 kB - -
@sentry/vue 37.51 kB - -
@sentry/vue (incl. Tracing) 54.33 kB - -
@sentry/svelte 29.55 kB - -
@sentry/remix (Remix 3 client bundle) 55.78 kB - -
CDN Bundle 31.22 kB - -
CDN Bundle (incl. Tracing) 51.98 kB - -
CDN Bundle (incl. Logs, Metrics) 33.46 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 53.92 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 74.21 kB - -
CDN Bundle (incl. Tracing, Replay) 89.56 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 91.53 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 95.72 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 97.71 kB - -
CDN Bundle - uncompressed 92.14 kB - -
CDN Bundle (incl. Tracing) - uncompressed 154.49 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 98.71 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 160.45 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 228.28 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 274.22 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 280.16 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 287.93 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 293.86 kB - -
@sentry/nextjs (client) 56.32 kB - -
@sentry/sveltekit (client) 51.87 kB - -
@sentry/core/server 40.59 kB - -
@sentry/core/browser 13.63 kB - -
@sentry/node 144.77 kB +0.01% +14 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 83.22 kB - -
@sentry/node - without tracing 93.32 kB +0.01% +9 B 🔺
@sentry/node - without channel injection 122.97 kB +0.01% +4 B 🔺
@sentry/aws-serverless 101.59 kB +0.01% +6 B 🔺
@sentry/cloudflare (withSentry) - minified 208.61 kB - -
@sentry/cloudflare (withSentry) 517.4 kB - -

View base workflow run

JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch from 13b24d7 to aa5e441 Compare September 23, 2026 05:27
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch from aa5e441 to 5638fe5 Compare September 23, 2026 07:57
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch from 5638fe5 to 1289b2e Compare September 23, 2026 09:12
JPeer264 removed this pull request from stack #24600 September 23, 2026 09:14
JPeer264 added this pull request to stack #24613 September 23, 2026 09:15
JPeer264 removed this pull request from stack #24613 September 23, 2026 09:51
JPeer264 added this pull request to stack #24633 September 23, 2026 09:51
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch from 1289b2e to 8128af7 Compare September 23, 2026 10:04
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch from 8128af7 to cbc7810 Compare September 23, 2026 12:25
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch 2 times, most recently from fc013ea to d067bb5 Compare September 23, 2026 13:31
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch from d067bb5 to a8dfce1 Compare September 23, 2026 16:19
JPeer264 removed this pull request from stack #24633 September 23, 2026 16:19
JPeer264 added this pull request to stack #24670 September 23, 2026 16:19
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch from a8dfce1 to b8b902f Compare September 23, 2026 16:40
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch from b55e889 to cb692c3 Compare September 24, 2026 14:19
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch 2 times, most recently from 7739767 to b91a8ac Compare September 25, 2026 15:31
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch from b91a8ac to 29eab42 Compare September 25, 2026 15:41
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch 3 times, most recently from dc84aab to 11257ca Compare September 27, 2026 13:47
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch 2 times, most recently from 82884ff to fb18e31 Compare September 27, 2026 14:57
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch 2 times, most recently from 6323645 to e3b7e76 Compare September 28, 2026 08:07
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch 3 times, most recently from 910920b to 7479e87 Compare September 28, 2026 09:38
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch 3 times, most recently from e781deb to 933683f Compare September 28, 2026 14:29
JPeer264 force-pushed the jp/react-router-8-cloudflare-into-framework branch from 933683f to 492d9ef Compare September 29, 2026 06:25
test(e2e): Move react-router-8-cloudflare MySQL tests into react-rout…
01cdbd7 
…er-8-framework

The Cloudflare variant of react-router-8-framework now covers what the
separate react-router-8-cloudflare app tested, so the MySQL route and tests
move over and the old app is removed. The tests now also run on Node, Bun
and Deno. The Worker is wrapped by `sentryCloudflareVitePlugin` instead of a
manual `withSentry` call.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
sentry-release-bot Bot mentioned this pull request Oct 7, 2026
Closed
47 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

chargome chargome approved these changes
isaacs Awaiting requested review from isaacs
andreiborza Awaiting requested review from andreiborza

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL