Skip to content

fix(tests): Remove nitro canary test job - #20473

Merged
logaretm merged 1 commit into
developfrom
awad/js-2265-nitro-3-canary-test-failed
Apr 24, 2026
Merged

fix(tests): Remove nitro canary test job#20473
logaretm merged 1 commit into
developfrom
awad/js-2265-nitro-3-canary-test-failed

Conversation

@logaretm

@logaretmlogaretm commented Apr 24, 2026

Copy link
Copy Markdown
Member

There is no stable stream of Nitro 3 canary builds yet, so no point in having this. I will add it back once we have a stable stream of builds, likely after the full v3 release.

I probably had it at one point then didn't remove it properly before merging #19224

Closes#20472

CopilotAI review requested due to automatic review settings April 24, 2026 01:28
@linear-code

Copy link
Copy Markdown

@logaretmlogaretm changed the title fix(tests): fix(tests): Remove nitro canary test jobApr 24, 2026

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the Nitro 3 canary E2E run from the canary GitHub Actions workflow, aligning CI coverage with the lack of a stable Nitro 3 canary build stream (per #20472).

Changes:

  • Drops the nitro-3 (canary) entry from the canary E2E matrix so it no longer runs in scheduled/CI canary tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@logaretm
logaretm requested review from JPeer264 and s1gr1dApril 24, 2026 01:30
@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize% ChangeChange
@sentry/browser25.88 kB--
@sentry/browser - with treeshaking flags24.35 kB--
@sentry/browser (incl. Tracing)43.8 kB--
@sentry/browser (incl. Tracing + Span Streaming)45.5 kB--
@sentry/browser (incl. Tracing, Profiling)48.73 kB--
@sentry/browser (incl. Tracing, Replay)82.98 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags72.5 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)87.67 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)99.93 kB--
@sentry/browser (incl. Feedback)42.7 kB--
@sentry/browser (incl. sendFeedback)30.55 kB--
@sentry/browser (incl. FeedbackAsync)35.55 kB--
@sentry/browser (incl. Metrics)27.16 kB--
@sentry/browser (incl. Logs)27.29 kB--
@sentry/browser (incl. Metrics & Logs)27.98 kB--
@sentry/react27.62 kB--
@sentry/react (incl. Tracing)46.05 kB--
@sentry/vue30.71 kB--
@sentry/vue (incl. Tracing)45.62 kB--
@sentry/svelte25.89 kB--
CDN Bundle28.57 kB--
CDN Bundle (incl. Tracing)46.08 kB--
CDN Bundle (incl. Logs, Metrics)29.95 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)47.12 kB--
CDN Bundle (incl. Replay, Logs, Metrics)68.92 kB--
CDN Bundle (incl. Tracing, Replay)83.14 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)84.17 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)88.61 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)89.69 kB--
CDN Bundle - uncompressed83.59 kB--
CDN Bundle (incl. Tracing) - uncompressed137.62 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed87.73 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed141.03 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed211.31 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed255.06 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed258.46 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed267.97 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed271.36 kB--
@sentry/nextjs (client)48.58 kB--
@sentry/sveltekit (client)44.22 kB--
@sentry/node-core58.35 kB+0.02%+9 B 🔺
@sentry/node175.68 kB+0.02%+27 B 🔺
@sentry/node - without tracing98.3 kB+0.02%+13 B 🔺
@sentry/aws-serverless115.33 kB+0.01%+11 B 🔺

View base workflow run

@logaretm
logaretm merged commit 358a49b into developApr 24, 2026
77 checks passed
@logaretm
logaretm deleted the awad/js-2265-nitro-3-canary-test-failed branch April 24, 2026 06:19
Sign up for freeto 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.

nitro-3 (canary) Test Failed

3 participants

@logaretm@nicohrubec