質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.51%

  • Git

    1289questions

    Gitはオープンソースの分散バージョン管理システム(DVCS)です。

  • GitHub

    771questions

    GitHubは、Gitバージョン管理システムを利用したソフトウェア開発向けの共有ウェブサービスです。GitHub商用プランおよびオープンソースプロジェクト向けの無料アカウントを提供しています。

  • GitLab

    127questions

    GitLabは Gitoliteをブラウザから管理できるようにする Rubyアプリケーションで、 GitHubのようなサービスをクローズドな環境に独自で構築できるように 公開されたものです。

gitでローカルのブランチをローカルのマスターにマージするのはNGですよね?

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 206

murabito

score 21

チーム開発においてのgitの使い方を勉強しているのですが、git flowみたいなやり方をベースにしている場合、ローカルのmasterをローカルのブランチにマージすることはあっても、ローカルのブランチをローカルのmasterにマージするっていうことは基本的にはないのですよね?(非推薦?)

リモートリポジトリの管理者みたいな人が、チームメンバーからプルリクエストされたブランチをリモートのマスターにマージすることになると思うので、チームの一人が勝手にローカルでブランチをマスターにマージしてたら、何かコンフリクトとか履歴がおかしくなったりとか、してしまいますよね?

「ローカルのmasterをローカルのブランチにマージすることはあっても、ローカルのブランチをローカルのmasterにマージするっていうことは基本的にはない」という認識であっているかどうかを確認したいです!

よろしくお願いいたします!

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+4

・プルリクエストの承認者と、承認者以外の人が分かれている
・マスターブランチへのマージは、プルリクエストの承認者しか行わない
という前提であれば、承認者以外の人がマスターブランチにマージすることは基本的にありません。

>チームの一人が勝手にローカルでブランチをマスターにマージしてたら、何かコンフリクトとか履歴がおかしくなったりとか、してしまいますよね?

勝手にマスターにマージさせるからといって、コンフリクトや履歴がおかしくなるのではないです。
コンフリクトは、修正した場所が被っていたら起こりますし、履歴はコミットが時系列に並ぶだけです。
ただし勝手にやることで、もしマスターブランチが本番用のブランチであれば、勝手にマージさせたことにより障害があった場合に責任が取れますか?ということです。

なお、この前提は、チームによって異なります。

私が業務上経験したことのあるチームでは、下記のようなケースがありました。
・完全にプルリクエストの承認者と、承認者以外の人が分かれているケース
・プルリクエストの承認者と、承認者以外の人は分かれているが、たまに承認者から承認者以外の人に「マージしておいて」と頼まれるケース
・誰もがプルリクエストの承認者になりうるため、プルリク承認の依頼がきたらコードレビューしてマスターにマージするケース

何が言いたいかというと、チーム開発は、会社の風土やチームの方針によってルールが変わるため、「ローカルのmasterをローカルのブランチにマージすることはあっても、ローカルのブランチをローカルのmasterにマージするっていうことは基本的にはない」と思い込むことは危険です。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/08 14:53

    ありがとうございます。大変参考になりました。

    キャンセル

+1

localかremoteか、というのは意味のない議論で、大切なのはbranchをmergeするに足る要件を満たしたか、merge権限を持っているか(開発メンバー内でのコンセンサス)です。

git-flowではdevelop branchからrelease branchを作ってこれをdevelopとmasterの両方にマージすることになると思います。

もう一度いいますが、localかremoteか、というのは意味のない議論です。git-flowにGithubのPull Requestは必須ではありません。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/08 14:54

    ありがとうございます。チームごとにまた違ってくるのですね。

    キャンセル

+1

ローカルのブランチをローカルのmasterにマージするっていうことは基本的にはないのですよね?(非推薦?) 

githubを使って開発をしている時、普通はPRを出してgithub上でマージしてます。

チームの一人が勝手にローカルでブランチをマスターにマージしてたら、何かコンフリクトとか履歴がおかしくなったりとか、してしまいますよね? 

誰かのローカルにあるmasterブランチに何かをマージして、pushせずに放置していて、別の誰かが同じファイルを更新してgithub上でmasterを更新したら、コンフリクトしますね。

「ローカルのmasterをローカルのブランチにマージすることはあっても、ローカルのブランチをローカルのmasterにマージするっていうことは基本的にはない」という認識であっているかどうかを確認したいです! 

あってると思います。

チームによっては、PRを使って他の人にレビューしてもらう必要がなさそうな修正は、直接masterを更新してるかもしれませんけどね。(例えば、コメントがタイポしてるとか)

とはいえ、PR出してレビューをしながら進める方がチームメンバーの納得感がありますし、基本的にPRを作って進めるというふうに考えておいて良いような気がします。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/08 14:54

    ありがとうございます。認識に間違いがさほどないようで安心しました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.51%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

  • 解決済

    【GIT】ブランチの作り方

    お世話になっております。Ezです。 Git ちょー初心者です。 Git のブランチについてご教示頂けないでしょうか? サルでもわかるGit入門を読んでいたのですが、わかりませんで

  • 受付中

    gitで同一ファイルを複数人で編集した場合の衝突

    gitで同一ファイルを複数人で編集した場合の衝突について質問です。 a.phpには既存コードが100行載っているとします。 a.phpをAさんがブランチhogeで編集していて、

  • 解決済

    gitで放置状態になったmasterブランチ

    gitで放置状態になっているmasterブランチがあり、他のブランチで開発が進んでおります。恐らくミスでそのような状態となっております。 masterが最新のコミットじゃないのは気

  • 解決済

    gitを使ったPHPサイト制作時のリポジトリ用フォルダーの場所

    複数人でPHPを使った動的なサイトを共同制作するためにGit(Bitbucket)を導入しようとしています。 PHPの作業環境はユーザーによってVagrantを使っている人もいれ

  • 解決済

    SourceTreeの基本的なことですが。

    windowsでSourceTreeを使っています。 別の作業者が、masterブランチで作業しており、途中から自分が加わることになったのですが、リモートから「新規/クローン

  • 解決済

    git mergeのやり直してについて教えてください

    現在、以下のような状態となっているのですが、aaaaaのマージをやり直す方法を教えていただけないでしょうか。リモートにプッシュ済みとなります。 * commit aaaa

  • 解決済

    github flowではmasterの変更を取り込む必要は無いのですか?

    GitHub flowでソース運用しています。 質問なのですが、 masterの変更が影響なければトピックブランチはmasterを反映する必要は無いのでしょうか?

  • 解決済

    gitのmergeでalready up to dateと表示される

     前提・実現したいこと ブランチAでファイルAを作成して、ブランチAでmasterにマージしてmasterブランチでファイルAの存在を確認したい。 回答よろしくお願いします

同じタグがついた質問を見る

  • Git

    1289questions

    Gitはオープンソースの分散バージョン管理システム(DVCS)です。

  • GitHub

    771questions

    GitHubは、Gitバージョン管理システムを利用したソフトウェア開発向けの共有ウェブサービスです。GitHub商用プランおよびオープンソースプロジェクト向けの無料アカウントを提供しています。

  • GitLab

    127questions

    GitLabは Gitoliteをブラウザから管理できるようにする Rubyアプリケーションで、 GitHubのようなサービスをクローズドな環境に独自で構築できるように 公開されたものです。