なぜpush時にデプロイ？

Question

GitのHook機能を使って，本番環境に自動でデプロイされるケースをネットでよく見かけます．　　
リモートにプッシュされた時点では，コードレビューはされておらず，それらに対してデプロイするのはまずいような気がするのですが，なぜそのようなことをするのでしょうか？

僕の素人考えでは，コードレビューがOKであれば，リモートでマージされるのでそのときにデプロイするのがいいように感じます...

Answer 1

基本的にhookはmasterブランチにかけますよね。
developやfeatureなどの開発ブランチにpushしたからといって本番環境にデプロイしているプロジェクトはないでしょう。
なのでmasterにマージされたらデプロイなので、よく見かけるものとMattさんの考えは同じことです。

Comments

[Matt]

開発ブランチがmasterにマージされる＝デプロイですか？
そういうことで考えると，自動でデプロイするとコードレビューをすっ飛ばすことになるのではないでしょうか．

[yuki-saito]

レビューをしないでmasterにマージすることなんて通常ありませんよ。
プルリクやマージリクエストをして、レビューして検証して全てが終わったものをmasterにマージするのが普通です。
Mattさんが思っている当たり前は、Gitでも当たり前です。
git-flowやgithub-flowについて調べてもらった方が良いかと思います。

[Matt]

いろいろと勉強不足でした．お答えいただき，ありがとうございました．

[yuki-saito]

お役に立ててよかったです＾＾