search

codeforjapan / mapprint

80 stars 58 forks source link

#459 の動作確認用PR #481

Open munierujp opened 9 months ago

munierujp commented 9 months ago

概要 | About

459 の動作確認のために、意図的にテストを失敗させるPRをつくります。

munierujp commented 9 months ago

@yuiseki masterブランチの内容でテストが実行されてしまっているっぽいですね……

halsk commented 9 months ago

@munierujp ありがとうございます。これ、Branch Protection の設定の問題ですかね？

munierujp commented 9 months ago

@halsk 自分もあまり詳しくないのですが、どちらかというとGitHub Actionsの設定かもしれません。

GitHub ActionsをCIに使うとき、マージ先のコードでビルトやテストが行われてしまう #GitHub - Qiita

@NEKOYASAN #464 でGitHub Actionsのトリガーに pull_request_target を指定されていますが、pull_requestではなにか問題があるでしょうか？

NEKOYASAN commented 9 months ago

Checksで利用するWorkflowだったので、Actionsをbase branch側のWorkflowを使うことでWorkflowの改ざんと回避を予防する意図がありましたが、@actions/checkout周りの修正を完全に忘れたままMergeしてますね...

pull_requestに変えてしまうか@actions/checkoutのrefをhead branchのshaにすることで解消できると思いますが、pull_requestに変えてしまうほうが都合良いですかね？