Skip to content

TC Merge Gate: treat draft tc/pr PRs as non-blocking - #7924

Draft
YeunjunLee with Copilot wants to merge 2 commits into
developfrom
copilot/fix-check-tc-prs-job
Draft

TC Merge Gate: treat draft tc/pr PRs as non-blocking#7924
YeunjunLee with Copilot wants to merge 2 commits into
developfrom
copilot/fix-check-tc-prs-job

Conversation

Copilot AI commented Sep 11, 2026

Copy link
Copy Markdown

Check TC PRs was failing even when testcase PRs were still in draft, which blocked engine PRs prematurely. This change narrows the gate so only open non-draft testcase PRs block merge.

  • Scope

    • Update .github/workflows/tc-merge-gate.yml gate logic for tc/pr-<PR_NUMBER> checks in:
      • CUBRID/cubrid-testcases
      • CUBRID/cubrid-testcases-private-ex
  • Behavior change

    • open + draft TC PR → pass (non-blocking, notice)
    • open + ready for review TC PR → fail (blocking, error)
    • no open TC PR / closed / merged → pass (unchanged)
  • Message alignment

    • Adjust failure and PR comment text to explicitly refer to non-draft open TC PRs.
if [ "${IS_DRAFT}" == "true" ]; then
  echo "::notice::${TC_REPO}: TC PR is open (draft): ${URL} — pass"
else
  echo "::error::${TC_REPO}: TC PR is open: ${URL}"
  OVERALL_PASS=false
fi

Co-authored-by: YeunjunLee <48234872+YeunjunLee@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job Check TC PRs TC Merge Gate: treat draft tc/pr PRs as non-blocking Sep 11, 2026
Copilot AI requested a review from YeunjunLee September 11, 2026 05:04
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