Skip to content

👷 Skip check-staging-merge for PRs targeting next major version branch#4181

Merged
thomas-lebeau merged 1 commit intomainfrom
thomas.lebeau/skip-staging-check-next-major
Feb 16, 2026
Merged

👷 Skip check-staging-merge for PRs targeting next major version branch#4181
thomas-lebeau merged 1 commit intomainfrom
thomas.lebeau/skip-staging-check-next-major

Conversation

@thomas-lebeau
Copy link
Copy Markdown
Collaborator

@thomas-lebeau thomas-lebeau commented Feb 13, 2026

Motivation

The check-staging-merge CI job verifies that a feature branch can be cleanly merged into the current staging branch. This check is only relevant for PRs targeting main (which go through the staging process). PRs targeting the v7 (next major) branch don't use staging, so this check is irrelevant and should be skipped.

Changes

  • Fetch PR info from GitHub API via fetchPR() to determine the target branch
  • If the PR targets NEXT_MAJOR_BRANCH (e.g. v7), exit early with a success message
  • Otherwise, continue with the existing staging merge check logic unchanged
  • Handles edge cases: no PR exists (continues check), NEXT_MAJOR_BRANCH not set (continues check)

Test instructions

  • PR targeting v7 → staging merge check is skipped
  • PR targeting main → staging merge check runs as before
  • Branch with no PR → staging merge check runs as before

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.
  • Updated documentation and/or relevant AGENTS.md file

@thomas-lebeau thomas-lebeau requested a review from a team as a code owner February 13, 2026 11:40
@thomas-lebeau thomas-lebeau merged commit 8af54f1 into main Feb 16, 2026
22 checks passed
@thomas-lebeau thomas-lebeau deleted the thomas.lebeau/skip-staging-check-next-major branch February 16, 2026 09:17
@github-actions github-actions bot locked and limited conversation to collaborators Feb 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants