コードレビューのやり方

[pss-aileen]

• チームでコードを書き始めた後、「どうやらレビューってやつをした方が良いらしい」くらいの若手に向けた資料です。

コードレビューやプルリク、
色々なことがはじめてなのでこういったことも話した方が良いかも？
次回のMTG時に共通認識が必要なものをまとめて話せれば良いなと思った。

[kazzyfrog]

ですね！
色々確認しつつ、まとめてMTGのアジェンダ的にしてもいいかも！

• コードレビュー
• プルリク
• イシュー

他にも、ドキュメントやイシューやディスカッションなど、どこをどの順番で見ればキャッチアップできるか、など？

[kazzyfrog]

コードレビューのやり方について、色々メモしていきますね！

• レビューの大まかな流れ
• まずセルフレビューするのも大事！
• より実践的なベストプラクティスとして、こんなまとめ記事がある

[kazzyfrog]

オープンソースプロジェクトの場合、PRは仕事ではなく善意から来る貢献だったりするので、
できるだけフランクにディスカッションするのが大事ですよね！

一方で、LGTMについての記事で、結構良いヒントが書いてありました：

レビュアーは「LGTM」が「このコードは我々の基準を満たしている」と確信できるように、レビューに十分な時間を費やすことが重要です。しかし個々のレスポンスはやはり理想的には迅速であるべきです。

PRを承認するかどうかは、コードやその他ドキュメントに対する自分たちの基準が大事になってきそうです！

[kazzyfrog]

@pss-aileen
もし良かったら、 #43 を練習がてらレビューしてみてください！
わざとtypoしてますw
あと、PRの内容も何やら悩ましい問題を抱えています！

レビューは、変更されたファイルの、気になる行を＋マークでコメント追加できます！
詳しくは、上に貼った「レビューの大まかな流れ」を参照してください！

もしわからない事があれば、サポートしますよ🙌

[pss-aileen]

わー！ありがとうございます！！
ちょっとたてこんでいるので今日夜、明日のどこかで確認させていただきます🙇
困ったら相談させてください！！