search

WORD-COINS / article-template

WORD記事の雛形。
BSD 3-Clause "New" or "Revised" License
8 stars 4 forks source link

Change revision link to head to its diff #68

Closed private-yusuke closed 2 years ago

private-yusuke commented 2 years ago

概要

Slack 通知の Revision の箇所で、当該 Revision におけるリポジトリのファイルを表示するページへリンクさせていたところを、そのコミットによる差分を表示するページへリンクさせるようにします。

動作確認

https://github.com/nektos/act を入れた自分の環境で $ act -s PDF_BUILD_NOTIFY_URL="ここに自分の個人 Slack の Webhook URL が入る" を実行し、Revision の部分のリンクから差分を表示するページへ飛ぶことができること

private-yusuke commented 2 years ago

動作確認の一つとして、この PR による通知（https://word-ac.slack.com/archives/CL40NDCMV/p1659931809411969）によって Revision のリンクから差分が見られるようになっていることがわかる

sksat commented 2 years ago

(こういうのはreview request発射しておくとよい again)

private-yusuke commented 2 years ago

review してほしい人を明示的に指定できるほど誰が担当しているのか把握していなかったので、ある程度泳がせたら適当に merge しようと考えていた（が、sksat に review してもらえたので助かります）

sksat commented 2 years ago

まあ曖昧に当該リポジトリにcommitしたことあるオタクに発射しとくのがいいんでは。誰も見てない中放置されてるのは単に腐ってるだけであんまり泳がせビリティがないため……

private-yusuke commented 2 years ago

実際前の PR も sksat に拾ってもらったから、出して放置するだけだと他の人の目に付かなかったのは納得してます。以前 commit したことある人をいきなり reviewer に指定してもいいんかな、と読んで最初は思ったが、考えているうちにそれで良さそうと思えたから次回からそうしてみます。🙏