Skip to content

ci: limit push trigger to main to avoid duplicate runs - #17

Merged
handlename merged 1 commit into
mainfrom
fix/nodejs-build-duplicated
May 17, 2026
Merged

ci: limit push trigger to main to avoid duplicate runs#17
handlename merged 1 commit into
mainfrom
fix/nodejs-build-duplicated

Conversation

@handlename

Copy link
Copy Markdown
Owner

Why this change is needed

PR branches triggered both push and pull_request events, causing the Node.js build workflow to run twice per Node version (20.x and 22.x). See https://github.com/handlename/obsidian-plugin-runestone/pull/16/checks

Approach

Limit the push trigger to the main branch only.

  • Pushes to PR branches now trigger only pull_request (once per Node version).
  • Pushes to main (after merge) trigger only push (once per Node version).

Notes for reviewers

Single-line YAML change; no behavioral impact on the build itself.

PR ブランチへの push で push と pull_request の両イベントが発火し、
各 Node バージョンが2回ずつ実行されていた問題を解消する。
push トリガーを main ブランチに限定することで、PR では
pull_request のみ、マージ後は push のみが実行される。

User request: lint.yml で Node.js build が各バージョン2回ずつ
実行されているのを、1回ずつに修正する。

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@handlename
handlename merged commit b0e916e into main May 17, 2026
2 checks passed
@handlename
handlename deleted the fix/nodejs-build-duplicated branch May 17, 2026 11:56
@handlename handlename mentioned this pull request May 17, 2026
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