Skip to content

ci: PR でも Build & Test を走らせる - #35

Merged
yasulab merged 1 commit into
mainfrom
ci-test-on-pull-request
Oct 26, 2026
Merged

yasulab merged 1 commit into
mainfrom
ci-test-on-pull-request

Conversation

@yasulab

@yasulab yasulab commented Oct 26, 2026

Copy link
Copy Markdown
Member

PR でも Build & Test(test.yml)を走らせます。

なぜ

  • Build & Test は main への push の時しか走らず、PR ではビルドもテストも確かめていなかった(PR で走るのは CodeQL だけ)
  • ci: Dependabot の PR を自動でマージする #34 で Dependabot の PR を自動でマージする。yasslab/dependabot_auto_merge は PR のチェックが通るのを待ってからマージするので、PR でビルドとテストが走らないと、確かめないままマージされる
  • main にマージされた後も、GITHUB_TOKEN でのマージは push の workflow を起動しないので、壊れていても翌朝の Daily Update まで気づけない

変えたこと

  • test.yml の on: に pull_request: を足す
  • ジョブ(名前は deploy だが、中身は jekyll build・rake test・rake test_scripts だけ)は contents: read のまま。スクリプトのテストは Mechanize の応答を差し替えていて、外部には出ない

マージ後に変わること

前 後
PR のチェック CodeQL だけ CodeQL と Build & Test
main への push Build & Test 変わらない

戻し方

  • この PR を revert する

Build & Test は main への push の時しか走らず、PR ではビルドもテストも確かめていなかった。
Dependabot の PR を自動でマージする時に、マージの前に通るのを待てるようにする。
ジョブは読み取りの権限だけで、スクリプトのテストも外部に出ない。
@yasulab
yasulab merged commit 8912b2f into main Oct 26, 2026
4 checks passed
@yasulab
yasulab deleted the ci-test-on-pull-request branch October 26, 2026 12:38
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