alunduil / alunduil/collection-json.hs

CI runs on every pull request regardless of target branch

オープン 初心者向け
#213 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
infrastructure
主要言語
Haskell
スター
3
フォーク
1
平均マージ
5時間 53分
マージ済み PR(30日)
63

説明

## Summary

Drop the `branches: [main]` filter from the `pull_request` triggers so a PR
targeting any branch gets the full check set.

## Motivation

`ci.yml:6-7`, `pre-commit.yml:6-7`, and `links.yml:6-7` all filter
`pull_request` to `branches: [main]`. A PR that targets anything else — a
stacked PR on top of a feature branch — runs zero checks. The default-branch
ruleset only covers `~DEFAULT_BRANCH`, so nothing gates that merge either.

Exposure is bounded: whatever eventually reaches `main` goes through a PR that
is checked. The cost is that a stacked branch can accumulate unverified commits,
and the failure surfaces late and all at once. alunduil-chezmoi leaves its
`pull_request` triggers unfiltered.

Low severity, one line per workflow.

## Scope

- Remove `branches: [main]` from the `pull_request:` trigger in `ci.yml`,
`pre-commit.yml`, and `links.yml`.
- Leave the `push:` triggers filtered to `main`, so a branch push with an open
PR still produces one run rather than two.

## Acceptance criteria

- [ ] A PR targeting a non-`main` branch runs the build matrix, `Hooks`, and
`Check links`.
- [ ] A push to a branch with an open PR produces one set of runs, not two.

## Additional context

Surfaced by a CI and checks audit.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

Start by inspecting the pull_request and push triggers at lines 6-7 of ci.yml, pre-commit.yml, and links.yml. Remove only the pull_request branch filters, preserve the push filters, and verify that non-main pull requests receive the build matrix, Hooks, and Check links while a branch push with an open pull request does not create duplicate runs.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
github-actions
領域
ci-cd
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
静か
明瞭さ
明確に書かれている
初心者へのやさしさ
88/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。