Githubでのレビュー？

質問

コミュニティーで知り合った方に完成したサンプルサイトを見て頂き、アドバイス助言等をいただく事になったのですが、

その方からGithubは使えますか？
レビューなどが楽になるので、使って下さいとのことでした。

その意味を理解できず困っています。
また、ここで意味するレビューとは何かと、どのように楽になるのかを教えて頂きたく質問した次第です。

何卒どうか助言を宜しくお願い致します。

回答No.1

dell_OK · ベストアンサー

うむ。
そのコミュニティーで知り合った方に教えていただくのが一番かと思います。

それはそれとして、Ｇｉｔをご存じなければ、まずはこちらなどからどんなものかを知っておくといいです。
https://backlog.com/ja/git-tutorial/
あと、レビューについてもこちらなどから。
https://docs.github.com/ja/free-pro-team@latest/github/collaborating-with-issues-and-pull-requests/about-pull-request-reviews
簡単に言うと、たぶん、どんな作り方をしているか、どんな手を加えたかなどを確認して、それが妥当であるかどうか検討するのがレビューだと思います。

楽になるのは、レビューする側もされる側もですね。どれどれどこをどうしたんだとする側は見るし、される側はこうしてけどどうでしょうか、って。
何を見てレビューするのかと言うと、Ｇｉｔはソース管理するもので、ソースの変更差分が出るので、それを見てレビューができます。
私が初めてレビューされた時はドキドキしました。「えー、変なことしてたらどうしよう」って。
この差分のおかげで、それまで無駄に修正履歴をソース内に記述していた文化もあって、そんなのをごっそりなくすことができます。間違った記述とかあったりして、「ウソ書くくらいなら、何も書かないで」って気持ちになります。
そんなのを一切書かなくて、単純にソースの差分を見る人間が見てこうなったんだって理解すればいいわけです。もし何かコメントを書きたいようなことがあれば、コミットコメントやプルリクエストに書いておけば、差分だけではわかりづらいようなことも、メッセージを残しておけます。

でと、ＧｉｔＨｕｂと言うのは、説明するより、こちらなどから。
https://www.i3design.jp/in-pocket/3111

差分やメッセージがどんな風に見れるのかと言うと、こんな感じ。
https://github.com/microsoft/vscode/pull/108403
レビューしてアドバイスもここに書けるので便利です。
差分だけ見ると、左にーや＋があるので、－の行が削除され、＋の行が追加された、ってことです。
https://github.com/microsoft/vscode/pull/108403/files
修正行はーと＋が連続していると思います。

私は個人で開発しているものはＧｉｔＨｕｂで管理しています。
バックアップも兼ねて。
あと、修正する前の状態に戻したい、ってことがよくあるので助かります。
仕事ではマイクロソフトのＤｅｖＯｐｓと言うを使っていて、これのＧｉｔでソース管理しています。
チームで開発した時に、レビューしたいのですが、なかなかメンバーのスキルが統一できないので、「どこなおした？」「変なにしてない？」って気になって仕方がないです。

手を加える時には、こまめにコミットして、何のための処理をどうしたって、小さな単位にしておいた方がいいです。
そのどれかはなかったことにして、他のだけ有効にしたい、ってこともできますから。
それに、もし複数の案件に対応したものがひとつのコミットになっていると、レビューする人間が困ります。
関連性のない修正が一度にされていると言うのは、とても見にくい差分になってくるからです。
差分の確認で重要なのは、「他のところは一切変更していない、だから他の処理で問題は起きにくい」と思えるかどうかだと、私は思っています。
当然、つついたところが他に影響しているかも知れないこともありますが、それまで大丈夫だったのだから、「おかしくなった原因はつついたここにある」って言いやすいわけです。

てな感じで、私が思うところのレビューのありかたとします。

お礼

ご丁寧でとても親切なご回答本当にありがとうございました！とてもわかりやすく、参考URLもとても勉強になりました。そして無事にレビューいただく事が出来解決しました！ありがとうございました！