Skip to content

ci: remove redundant main push triggers from build and ci workflows#16

Merged
tomatoaiu merged 1 commit intomainfrom
ci/slim-workflows
Feb 11, 2026
Merged

ci: remove redundant main push triggers from build and ci workflows#16
tomatoaiu merged 1 commit intomainfrom
ci/slim-workflows

Conversation

@tomatoaiu
Copy link
Copy Markdown
Owner

Summary

  • build.yml: push: mainpull_request: main に変更
  • ci.yml: push: main トリガーを削除(pull_request のみに)

Motivation

main へのマージ時に Build / CI が走るのは無駄:

  1. PR で検証済み — fmt, clippy, ビルドすべて PR 段階で通過している
  2. Release と Build の重複 — タグ push 時に release.yml が同じ Windows ビルドを実行するため、build.yml が main push でも走ると同一ビルドが2重に走る
  3. 決定的チェック — fmt/clippy は同一コードなら結果が変わらない

Before

Workflow PR main push tag
Build - Yes -
CI Yes Yes -
Release - - Yes

After

Workflow PR main push tag
Build Yes - -
CI Yes - -
Release - - Yes

Test plan

  • PR 作成時に Build / CI が発火することを確認
  • main マージ時に Build / CI が発火しないことを確認

🤖 Generated with Claude Code

Build and CI workflows now run only on pull requests. Main merge no
longer triggers duplicate runs since PRs already validate the same code.
Release workflow (tag-triggered) remains the sole build on main.

Co-Authored-By: Claude <noreply@anthropic.com>
@tomatoaiu tomatoaiu self-assigned this Feb 11, 2026
@tomatoaiu tomatoaiu merged commit 0aff91b into main Feb 11, 2026
3 checks passed
@tomatoaiu tomatoaiu deleted the ci/slim-workflows branch February 11, 2026 02:26
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