DevhubHook icon indicating copy to clipboard operation
DevhubHook copied to clipboard

通知時にメンバ宛のメッセージにしたらレビューしてくれそうで良いかも

Open volpe28v opened this issue 10 years ago • 6 comments

コードレビューを促進するために、 push通知を特定のメンバの中からランダムで宛先に指定したら面白いかも

全部見ようとしたら大変だけど、少なくとも自分宛てに来たpushくらいは目を通そうかなと思う気がする。

volpe28v avatar Aug 14 '14 14:08 volpe28v

ランダムであなた一人にしか届いてない旨が強調されないと, だれも目を通さない事態発生するかも(笑

mocomaruz avatar Aug 14 '14 15:08 mocomaruz

ランダム宛先は、 @hogehoge で指定するだけなので、みなさんに push 通知は行くイメージです。

具体例としては  xxxx が push されました。 @hoge さんレビューお願いします!

というように緩い感じで必ずしもレビューしなくてもいいですけど、してくれたらいいな的な。 おそらく現状でも実際に進んでコードに目を通している人は限られているので、こういう仕掛けでコードを見る起爆剤になればいいかなと。

volpe28v avatar Aug 14 '14 15:08 volpe28v

あ, なるほど 特定の人のみにpush がいくのではなく, だれだれさん見てみて〜ってのを付け加えるわけですな.

いいですね. アンテナ立ってるひとはちょいちょい見てるけど見ない人は見ないですからねー

mocomaruz avatar Aug 14 '14 15:08 mocomaruz

そうです! なんとなく、push通知が流れるのに慣れてきてるので、空気みたいにスルーする感じになっちゃったらもったいないかなと思ったのです。

http://blog.jnito.com/entry/2014/08/13/084412 この辺りを読んでて「やっぱりコードレビューって大事だよなぁ」って思ったので大脳経由せずに issue 上げてしまいました :sweat_smile: @hashrockさんすみません!

volpe28v avatar Aug 14 '14 15:08 volpe28v

よいですね! 宛先メンバのマスターはjsonか何かで予め定義しておくイメージでしょうか?

もしくは、@randomとかを通知メッセージに入れておけばDevHub上のDavidが ログイン中のメンバから一人指名するとかもいいかもですね。

hashrock avatar Aug 18 '14 15:08 hashrock

最初はjsonか何かで予め渡しておくイメージでしたが、@random 機能面白いですね! くじ引き的な使い方も出来て他の用途にも応用できそうですね。 すごくアリな気がします!

volpe28v avatar Aug 18 '14 15:08 volpe28v