Skip to content

fix(ci): prepare main branch migration - #6066

Merged
killagu merged 1 commit into
nextfrom
fix/main-branch-ci
Oct 6, 2026
Merged

killagu merged 1 commit into
nextfrom
fix/main-branch-ci

Conversation

@killagu

@killagu killagu commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Renaming the default branch from next to main would leave CI and E2E push/pull-request triggers matching only next. Allow both main and next before the rename, and default the manual release branch input to main. Keep existing release ref validation and add coverage showing the default branch passes that guard.

Validation: all 10 release/CI tests passed; YAML checks verified both push and pull-request filters for CI/E2E; git diff --check passed. This change does not rename branches, change repository settings, dispatch releases, or publish packages. Merge before the branch migration.

Summary by CodeRabbit

  • Chores
    • Automated checks now run for the main and next branches as configured; end-to-end checks continue to skip Markdown-only changes.
    • Manual releases now default to main, while both main and next remain available.
    • These updates align routine validation and release defaults with main while retaining support for next.

Copilot AI balanced review requested due to automatic review settings October 6, 2026 14:05

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying egg with  Cloudflare Pages  Cloudflare Pages

Latest commit: d45200c
Status: ✅  Deploy successful!
Preview URL: https://4def59f9.egg-cci.pages.dev
Branch Preview URL: https://fix-main-branch-ci.egg-cci.pages.dev

View logs

@coderabbitai

coderabbitai Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 23ff995a-39d6-49a7-a98e-03e999e4ed02
📥 Commits

Reviewing files that changed from the base of the PR and between 32018d5 and d45200c.

📒 Files selected for processing (4)
  • .github/workflows/ci.yml
  • .github/workflows/e2e-test.yml
  • .github/workflows/release.yml
  • scripts/test/release.test.js

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 7 remain after this review.


📝 Walkthrough

Walkthrough

The CI and end-to-end workflows update their branch triggers. The manual release workflow defaults to main, and its test verifies the configured default.

Changes

Branch workflow updates

Layer / File(s) Summary
CI and end-to-end triggers
.github/workflows/ci.yml, .github/workflows/e2e-test.yml
CI push and pull-request triggers now target main and next. End-to-end push and pull-request triggers now target main, with next retained for pushes.
Release branch default
.github/workflows/release.yml, scripts/test/release.test.js
The release workflow defaults to main. The test checks that default and uses it in a successful branch-ref case.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Feature

Suggested reviewers: fengmk2

Merge Risk: ⚪ Minimal · up to d4520

Both main and next remain covered by the workflow triggers, and the release default is tested. No actionable merge-blocking risk is established.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the CI and release configuration changes that prepare for the main branch migration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Commit to this branch
  • Create a new PR
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying egg-v3 with  Cloudflare Pages  Cloudflare Pages

Latest commit: d45200c
Status: ✅  Deploy successful!
Preview URL: https://c5d9e4c6.egg-v3.pages.dev
Branch Preview URL: https://fix-main-branch-ci.egg-v3.pages.dev

View logs

@codecov

codecov Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.95%. Comparing base (32018d5) to head (d45200c).

Additional details and impacted files
@@            Coverage Diff             @@
##             next    #6066      +/-   ##
==========================================
+ Coverage   83.59%   83.95%   +0.36%     
==========================================
  Files         731      731              
  Lines       22609    22609              
  Branches     4554     4554              
==========================================
+ Hits        18899    18981      +82     
+ Misses       3206     3134      -72     
+ Partials      504      494      -10     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@killagu
killagu merged commit 833c645 into next Oct 6, 2026
37 of 38 checks passed
@killagu
killagu deleted the fix/main-branch-ci branch October 6, 2026 14:20
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