vim-jp / ekiden

vim-jp記事執筆企画「駅伝」の管理リポジトリ
https://vim-jp.org/ekiden/
Creative Commons Attribution 4.0 International
49 stars 19 forks source link

Duplicate runs of checking workflow by `check-article.yml` #433

Closed peacock0803sz closed 1 month ago

peacock0803sz commented 1 month ago

411, #312, #413, #420 で再現している模様、おそらく#411 より前は発生していないと思われる。

https://github.com/vim-jp/ekiden/blob/ee84d80e45952a476dccadca5095ddb9cc32408c/.github/workflows/check-article.yml#L8-L10

このあたり↑で2回発火しているのは良いはず。

https://github.com/vim-jp/ekiden/blob/ee84d80e45952a476dccadca5095ddb9cc32408c/.github/workflows/check-article.yml#L69-L73

本当の原因はこっち↑の if が動作していないからか?

Screenshots in Reproductions (#420)

CleanShot 2024-05-23 at 14 19 15

Workflow Runs (#420)

peacock0803sz commented 1 month ago

share: @thinca

mityu commented 1 month ago

おそらくhttps://github.com/vim-jp/ekiden/issues/411 より前は発生していないと思われる。

@... 投稿を確認できました〜」が2回投稿される、という話であれば割と前から発生していて、多分 https://github.com/vim-jp/ekiden/issues/272 この辺りが境目なんじゃないかなーと。

peacock0803sz commented 1 month ago

おそらく#411 より前は発生していないと思われる。

@... 投稿を確認できました〜」が2回投稿される、という話であれば割と前から発生していて、多分 #272 この辺りが境目なんじゃないかなーと。

@mityu おー、ほんとだ。確認ありがとうございます!!

k-takata commented 1 month ago

https://github.com/vim-jp/ekiden/commit/c83a2ca03d06a234f90b5595d587acbe884667d2 completed で close されている場合、そのまま残りの処理を行うので、2回投稿されるのではないかと。