Skip to content

fix(ci): remove PR trigger from release workflow to prevent duplicate execution - #191

Merged
baleen37 merged 2 commits into
mainfrom
fix/remove-pr-trigger-from-release-workflow
Jan 27, 2026
Merged

fix(ci): remove PR trigger from release workflow to prevent duplicate execution#191
baleen37 merged 2 commits into
mainfrom
fix/remove-pr-trigger-from-release-workflow

Conversation

@baleen37

@baleen37 baleen37 commented Jan 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • PR 트리거 제거로 중복 실행 방지
  • 이제 main에 push할 때만 workflow 실행

Test plan

  • YAML 문법 검증 완료
  • BATS 테스트 101/101 통과
  • 워크플로우 로직 검증 완료

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Streamlined the release workflow automation to improve efficiency and consistency of the release process.

✏️ Tip: You can customize this high-level summary in your review settings.

… execution

The pull_request trigger was causing the workflow to run on both:
1. PR merge (pull_request event)
2. The resulting push to main (push event)

This created duplicate executions. Now the workflow only triggers on
push to main branch, which covers both direct pushes and PR merges.

Removed:
- pull_request trigger (lines 7-10)
- Conditional PR merge logic in Release step

The workflow now:
- Triggers only on push to main
- Runs semantic-release to update version files
- Creates a release PR if changes are detected

Co-Authored-By: Claude <noreply@anthropic.com>
@baleen37
baleen37 enabled auto-merge (squash) January 27, 2026 03:44
@coderabbitai

coderabbitai Bot commented Jan 27, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

Simplifies the release workflow by removing PR merge event gating and conditional branching logic. The workflow now unifies all release steps under a single push-to-main path, eliminating the distinction between PR merge and push events while retaining semantic-release and PR automation.

Changes

Cohort / File(s) Summary
Release workflow simplification
.github/workflows/release.yml
Removed PR merge conditional checks and unified push-to-main logic. Consolidated semantic-release, versioning, commit, branch/PR creation, and auto-merge steps into a single streamlined flow without event-based branching.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

released

Poem

🐰 The workflow hops with clarity and grace,
No branching conditions to slow the race,
Push to main—one golden path to take,
Unified release, no choices to make!
Semantic versioning hops so true,
A simpler flow for all to pursue!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing the PR trigger from the release workflow to prevent duplicate executions, which is the core focus of this changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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 and usage tips.

@baleen37
baleen37 merged commit d2f371e into main Jan 27, 2026
2 checks passed
@baleen37
baleen37 deleted the fix/remove-pr-trigger-from-release-workflow branch January 27, 2026 03:53
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