Skip to content

ci: advisory Development-merge check for PRs to main#235

Open
rlorenzo wants to merge 1 commit into
mainfrom
ci/development-merge-check
Open

ci: advisory Development-merge check for PRs to main#235
rlorenzo wants to merge 1 commit into
mainfrom
ci/development-merge-check

Conversation

@rlorenzo

@rlorenzo rlorenzo commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds an advisory, non-blocking CI check that nudges toward the team workflow of routing branches through Development before main.

Behavior

Runs on PRs targeting main:

  • Looks for a merged PR with base Development and head = this branch.
  • If found, the check passes quietly.
  • If not, it emits a warning annotation ("Branch X has not been merged into Development yet...") but the job still succeeds.

It never fails, so it does not block merging and does not need to be added to branch protection. It is purely informational.

Adds a non-blocking workflow that warns when a PR into main is opened whose
branch has not yet been merged into Development, matching the team workflow of
routing branches through Development before main. It emits a warning annotation
but never fails, so it does not block merging — purely informational.

"Merged to Development" is detected via a merged base=Development pull request
(robust to this repo's squash merges) rather than commit ancestry, which a
squash would not preserve.

Claude-Session: https://claude.ai/code/session_01DtQRoktAcsjxGQs5e1aEoF
@codecov-commenter

Copy link
Copy Markdown

Bundle Report

Bundle size has no change ✅

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 44.51%. Comparing base (1d971df) to head (5580411).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #235   +/-   ##
=======================================
  Coverage   44.51%   44.51%           
=======================================
  Files         895      895           
  Lines       51760    51760           
  Branches     4827     4827           
=======================================
  Hits        23041    23041           
  Misses      28149    28149           
  Partials      570      570           
Flag Coverage Δ
backend 44.67% <ø> (ø)
frontend 41.32% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

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