チーム開発でのGIT操作でのトラブル

Question

チーム開発で起こったgitのトラブルでの質問です。

まず自分のブランチで作業をしていて、
１つのファイルのコードを後で確認したい為、
コードの上にコメントアウトの文字を
記入していました。
(ファイルAとします)

その後PMから他のチームメンバーの変更を
マージしたから、その変更をpullしてくださいと連絡があり、

git checkout develop

コマンドを実行したら、

error: Your local changes to the following files would be overwritten by checkout:
	src/app/Http/Controllers/XXXXX.php
Please commit your changes or stash them before you switch branches.
Aborting

とエラーが出現したので、
ファイルAの変更を退避するために、

git stash -u

を実行。その後、

git checkout develop
git pull
git checkout 自分のブランチ
git merge develop
git add~pushまで

を行いプルリクを投げたのですが、
PMからは文言的には

merge branch develop into ブランチ

っていうのが残ってるのが正しいと
言われている状況です。
チームメンバーも質問もできない状況なので、
今の状況になった原因などご教授いただけないでしょうか？

Comments

[Supernove]

逆にdevelopブランチにマージしたときの実際のコミット文言はなんでしたか？

[68user]

feature ブランチに "merge branch develop into ブランチ" というマージのログが残っているはずということですよね?
手順は問題ないように見えますけどね。
で、実際はどういうログが残っていて、develop の修正は feature に正しく取り込まれているんですか?

[hoshi-takanori]

自分のブランチに git merge develop する前にコミットしてなければ fast forward merge になって「merge branch develop into ブランチ」のログが残らない可能性はありますね。でも、それで何か問題があるのかな…。

Answer 1

merge branch develop into ブランチ

この文言はいわゆる「マージコミット」として作成されるコミットのデフォルトの文言です。
もしPMからの指示が「他のブランチからコミットを取り込んだ場合は必ずマージコミットを作成せよ」というものであるなら、 git merge コマンドに --no-ff オプションを追加するか、config に追加しましょう。

https://qiita.com/nog/items/c79469afbf3e632f10a1

本質的にはマージコミットが必要ということは無いのですが、PMの管理上存在することが望ましいと思うのであれば改めてチーム全員できちんと合意を取るべきでしょう。