DevhubHook
DevhubHook copied to clipboard
通知時にメンバ宛のメッセージにしたらレビューしてくれそうで良いかも
コードレビューを促進するために、 push通知を特定のメンバの中からランダムで宛先に指定したら面白いかも
全部見ようとしたら大変だけど、少なくとも自分宛てに来たpushくらいは目を通そうかなと思う気がする。
ランダムであなた一人にしか届いてない旨が強調されないと, だれも目を通さない事態発生するかも(笑
ランダム宛先は、 @hogehoge で指定するだけなので、みなさんに push 通知は行くイメージです。
具体例としては xxxx が push されました。 @hoge さんレビューお願いします!
というように緩い感じで必ずしもレビューしなくてもいいですけど、してくれたらいいな的な。 おそらく現状でも実際に進んでコードに目を通している人は限られているので、こういう仕掛けでコードを見る起爆剤になればいいかなと。
あ, なるほど 特定の人のみにpush がいくのではなく, だれだれさん見てみて〜ってのを付け加えるわけですな.
いいですね. アンテナ立ってるひとはちょいちょい見てるけど見ない人は見ないですからねー
そうです! なんとなく、push通知が流れるのに慣れてきてるので、空気みたいにスルーする感じになっちゃったらもったいないかなと思ったのです。
http://blog.jnito.com/entry/2014/08/13/084412 この辺りを読んでて「やっぱりコードレビューって大事だよなぁ」って思ったので大脳経由せずに issue 上げてしまいました :sweat_smile: @hashrockさんすみません!
よいですね! 宛先メンバのマスターはjsonか何かで予め定義しておくイメージでしょうか?
もしくは、@randomとかを通知メッセージに入れておけばDevHub上のDavidが ログイン中のメンバから一人指名するとかもいいかもですね。
最初はjsonか何かで予め渡しておくイメージでしたが、@random 機能面白いですね! くじ引き的な使い方も出来て他の用途にも応用できそうですね。 すごくアリな気がします!