承認ではなくて、よさそう、と思って暮らしている - hitode909の日記

普通に書いたdiffは、関心がさまざまなところに散らばっていたたり、書きかけだったりで、意味のまとまりがないもので、それを整形して、説明を書いたものがPull Requestであり、コードレビューは、そのまとまりごとに、他人から見て理解可能であるという承認する行為、という理解をしていた。 なので、レビューを通すことは、動くことに賭けて、以後、動かなかったら責任を取る、みたいなイメージはあまり持っていなかった。 レビュワーの責任をどこまでと規定するか考えて、責任が大きい順に並べていくと レビューを通した以上、以後は私の責任です、という態度 職人魂を感じる 見たところよさそうに思いました、という態度 通りすがり風情を感じる まったくの無責任なので、工数最小化のために何が来てもapproveする、という態度 やっつけ仕事 かるぱさんのチームでは1になっているのかな(追記)こうなっている、というこ

[karupanerura]

ほとんど同じような感覚で仕事してて、ただそんな具合で暮らしているなかときどき1だと思い込んでいそうな人に出会うことがあって、なにがそのような考えをもたらすのかを考えたうちの一部が件のエントリなのでした

[Sixeight]

レビューを通した以上、以後は私とあなたの責任です、というつもりでやっていた。見たところよさそうだとコードを書いた人が必要以上に責任を感じてしまいそうと思ってしまう。

[likk]

これについてはウチのチーム（弊社じゃなくてあくまでチーム）は２に近く一部において３かな。チームの中で約束は必要だと思う。[レビューされてないけどマージした]という意味のラベルも用意してある。

[onk]

2 がバランス良いと思って暮らしている。関わった人全員が薄く責任を負っている、個々の事象よりも全体の SLO についてチームや TL が責任を負う、という設定が好きです。

[mangano-ito]

コードレビューするときは通常(1)の気持ち．される側のときはどれでもよい気持ち．神経質なので壊れてもいいワークフローでも同じ基準でいそう．

[koogawa]

ワシは 2 のスタンス

[TrinityT]

自分も通常は2のスタンスが円滑にチームを回す落とし所だと思うな。課金周りなど重要な部分は1のスタンスであろうとしているが。

[yujiorama]

"お互い歩み寄って、最速で収束させるぞ！みたいな気持ちが大事だと思う。"