セルフレビューの記事

読んだ。

セルフレビューのコツ ○

そもそも「セルフレビュー」を明確にやっているという感じではなかったので、この記事を読んでちょっと恥ずかしくなった。セルフレビュー、やります……。

記事ではプルリクエストのコメントを使ってセルフレビューするというコツが紹介されているが、この方法をすでにチームメンバーの方々がやっているなとふと思った。

正確にはセルフレビューとまでは行かないかもしれないけど、僕がレビューするときに「分かりにくそうなところ」「ぱっと見て違和感があるかもしれないけど意図してこう書いた」「このプルリクエストでは対応しないが今後別のプルリクエストで対応する」などのコメントをよく見かけることがある。

こういうコメントがあるとレビューする側の負荷がシンプルに減るし、何より書いてもらったコードが読みやすくなると思う。自分も真似していきたい。

自分が上げたプルリクエストには、「ここの実装方法でもっと良い方法があれば教えてください」といったコメントをすることはあるけど、それ以外の部分で ↑ のようなチームメンバーの方々がやっているようなことができていなかったので、意識するようにしたい。