Skip to content

ci: retarget non-release PRs from main to develop; test PRs into develop - #20

Merged
levz0r merged 1 commit into
developfrom
ci/pr-base-guard
Oct 25, 2026
Merged

levz0r merged 1 commit into
developfrom
ci/pr-base-guard

Conversation

@levz0r

@levz0r levz0r commented Oct 25, 2026

Copy link
Copy Markdown
Owner

main only receives release PRs from develop. The default branch is now develop, so new PRs target it by default; this catches anyone who picks main explicitly.

  • New pr-base-guard.yml: on pull_request_target (opened/reopened/edited) against main, any PR whose head isn't levz0r/develop is retargeted to develop with a short comment. It never checks out or runs PR code; untrusted fields are used only in the if: expression.
  • test.yml: also run on PRs into develop, which previously got no CI (Renovate's Update non-major dependencies #15/Update major dependencies (major) #16, and contributor PRs once retargeted).

Note: pull_request_target runs the workflow from the base branch, so the guard only takes effect once this reaches main in the next release PR.

🤖 Generated with Claude Code

main should only receive release PRs from develop. Add a
pull_request_target workflow that retargets any other PR opened
against main to develop and leaves a comment. It never checks out
PR code, so it is safe for fork PRs.

Also run the test workflow on PRs into develop, which until now got
no CI at all (Renovate PRs, and contributor PRs once retargeted).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@codecov

codecov Bot commented Oct 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (e0f9905) to head (a0eaad9).

Additional details and impacted files
@@            Coverage Diff            @@
##           develop       #20   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            4         4           
  Branches         1         1           
=========================================
  Hits             4         4           

☔ 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.

@levz0r
levz0r merged commit 2b69483 into develop Oct 25, 2026
4 checks passed
@levz0r
levz0r deleted the ci/pr-base-guard branch October 25, 2026 09:51
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.

1 participant