githubでpull requestしたものが消える

Question

原因は私の理解が浅いせいなのだと思うのですが、
何故こうなったのかわからないので理由が推測出来る方は教えて下さい。

ある仕事用のリポジトリでmasterからブランチを作って（ブランチAとします）
修正を加えてマージ→pullrequest→master反映としたのですが、
その次に別の人がコミットしたファイルからは私の修正が消えていました。
その方がコミットする時に消しこむような修正をしたのかと思い、コミットログを見たのですが、
特に私が加えたファイルを消したようなログはありません。
元からコミットなどなかったかのような扱いになっています。
私のコミットログにはきちんとファイルは加えられており、
断面を見てもファイルは残っています。
今回は手動でもう一度ファイルを入れなおして終わったのですが、
またやってしまいそうで原因がわからず不安です。

ブランチA作成            マージ  他の方のコミット
→   ● →       …      → ● →  ● →
     ↓                     ↑  （以降私がコミットしたファイルなし）
     ↓ → → → →● → →  ↑
           ブランチAで修正      ブランチAで修正

Answer 1

お二方ご回答ありがとうございました。
あれから調べた結果、どうやら私ともうひと方が同じmasterのバージョンに対して、
修正コミットとしたのが問題だったようです。
同じバージョンに対してコミットしたので上書き扱いで私のものが消えたようです。
masterバージョン１→masterバージョン２（１ベース 私のブランチAからのコミット）→masterバージョン２’（１ベース 他の方のコミット）→（その後はバージョン２’が正となる）

先にコミットが入っている場合は、pullしないかぎり、pushできないものだと思っていたので、
混乱していました。
数人ですので、次からは、大きなコミットをする直前には、
きちんと宣言して、上書き合戦が起こらないように気をつけようと思います。

Comments

[takotakot]

push ではなく、push -f をされたということですね。pull して、既に push されている コミット と整合性をとらないと、このようなことが起きることがあります。

Answer 2

同じく勉強中のものです。

作業者の方は、修正更新後のmasterからローカルリポジトリを作成してますか。
もし、修正前のmasterからブランチを切って作業されているとしたらコミットしても古いデータとなってしまいます。

運用方法の全貌は見えませんが、下記のような感じで修正しないでしょうか。
１．(管理者)master更新、報知
２．(作業者)新masterをローカルのmasterにpull+マージ
３．(作業者)pullしたmasterと作業用のブランチAをマージ
４．(作業者)ブランチAにて作業を進める

以上でございます。

Comments

[shichiria]

今のプロジェクトだと数人なので管理者がいない状態で、
小さい修正→直接マスターに、大きい修正→ブランチを切って修正、
という程度しか決まっていません。
なので自分の修正は自分でmasterへのマージ作業まで行っています。

Answer 3

1. Aブランチで作業する（適宜ステージングする）
2. Aブランチをコミットする

この状態なったらリモートにpushできますよね。
作業中、基本的にマージはしませんよ。

で、
リモートにpushしたということは、githubにも反映されます。
その反映されたpushされたコミットを、みんなに見てほしいわけです。
ようするにみんなにこのコミットをpullしてほしい。
なので、ここで初めてpull requestを出すということになります。

pull requestをみんながみて、問題ないと思ったら（ルールは現場によって違うけど、いいねを3つもらったら、とかいろいろ）そこで初めてマージします。

pull request前にマージしたのはなんでですかね？
（みんながそうしてたから、そうしろといわれたから、とかなんでもいいですよ）

Comments

[shichiria]

小さなプロジェクトなのでPullRequestを出して取り込むのも自分でやっています。
（修正を入れること自体は事前に全員に通知しています）
gitの良さを生かしていないのはわかっていますが、メンバー間の利便性が優先されています。
マージをしたのはconflictが発生したので、それを解消するために一旦masterの修正を取り込んだためです。その後、pullrequestを出して、承認ボタンを押しました。

[ogaaaan]

小さいプロジェクトだとなかなかgitの良さが活かせない事ありますよねー。
だいたいgitってコマンド多すぎるんですよwww

閑話休題。

これはあくまで想像なのですが、Aブランチを他の方が使った可能性はないですかね。
その人はあなたの更新をpullして取り込まないままpushして、上書き合戦になってしまったと。