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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Git

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

GitHub

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

Q&A

解決済

2回答

598閲覧

git、githubでのコードレビューの回し方

sequelanonymous

総合スコア123

Git

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

GitHub

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

0グッド

0クリップ

投稿2018/09/13 08:20

先にAというファイルをpushしてレビューしてもらってる間にBというファイルを修正したい。けど、Aファイルの再修正の依頼が来る可能性があるため、Bファイルの修正を先に進めてしまうと、Aファイル修正後のコミットにBファイルの修正が含まれてしまう可能性がある。つまり、レビューしてもらっている間のBファイルの修正分ってどういうふうにいつも対処すべきでしょうか?

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

B機能はA機能に依存しますか?
関係ない別物であるならばA機能からブランチを生やすのではなく、
masterブランチからB機能用のブランチを作成して作業してください。

B機能がA機能に依存している場合は、
B機能はA機能から生えているのが自然です。
別にそのまま作業すれば良いと思います。


さて、その時に困るのはこういう事じゃないですか?

A機能に依存するB機能を修正する為に、Aブランチから派生して作業を開始したが、
A機能がレビューを取り込んで更新したらコミット履歴的に入れ子になるんじゃない?

同じファイルをB機能でガッツリ触っていればコンフリクトが発生するでしょうけど、
git rebaseでA機能の追加修正のコミット履歴を先に差し込む事が可能です。

bash

1# 作業中のブランチを確認 2$ git branch 3 master 4 A 5* B 6 7# A機能の修正を行う為に、B機能を一次保存 8$ git commit -m "tmp" 9 10# A機能にcheckoutして修正 11$ git checkout A 12$ git add a.file 13$ git commit -m "A機能のレビュー取り込み修正対応" 14$ git push origin A 15 16# B機能のコミット履歴の前にAの修正内容を差し込む 17$ git checkout B 18$ git rebase A

投稿2018/09/13 09:03

miyabi-sun

総合スコア21158

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

Bの修正は別ブランチで行う事が正解なように感じます。

投稿2018/09/13 08:23

n_takapyon

総合スコア443

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問