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

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

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

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

Q&A

解決済

2回答

383閲覧

github flowでpush後に不具合に気付いたら

退会済みユーザー

退会済みユーザー

総合スコア0

Git

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

0グッド

1クリップ

投稿2018/06/06 13:53

masterに取り込まれた場合は修正ブランチをきれば良いでしょうか?
また、取り込まれる前はpushしなおせば良いのでしょうか?

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

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

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

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

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

guest

回答2

0

ベストアンサー

何のブランチをPushしたかにもよりますが、
master以外のブランチならセーフ、masterなら何やってんのという話になります。
プルリクエストのレビューや自動テストを使って、確実にmasterには動くと保証出来る状態を保ちましょう。

masterに取り込まれた場合は修正ブランチをきれば良いでしょうか?

1人プロジェクトならこっそりmasterを直で触って修正すれば良いと思いますし、
チームを組んで開発しているならごめんねを込めたプルリクエストを出すという正規の手順でmasterを修正しなおすべきでしょう。
バージョンを確定して後から追いやすくする目的で「タグ」機能を使いましょう。

また、取り込まれる前はpushしなおせば良いのでしょうか?

master以外のブランチAをpushし、プルリクエストをmasterに投げるとします。
この状態で、Aに更にコミット履歴を積み上げてpushすると、プルリクエストに今追加したコミット履歴を含めた状態のプルリクエストに進化します。

なのでmasterに取り込まれる前ならセーフ。
プルリクエストで何をやりたいのかというと、他人の目でレビューがしたいわけです。

  • 実装者: こんなパッチ作ったよ取り込んでくれる?
  • レビュワー: この行バグ出るんじゃね?不要なコメント見つけたから削除よろしく。この行コーディング規約無視してるぞ
  • 実装者: ごめんごめん直しとくわ
  • 実装者: パッチ修正してpushしたから再度確認お願いね
  • レビュワー: 確かに指摘箇所全部直ってるな、よしマージ実行!

こういったやり取りがスムーズに行えるようになっています。

投稿2018/06/06 14:28

miyabi-sun

総合スコア21158

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

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

退会済みユーザー

退会済みユーザー

2018/06/06 14:30

よくわかりました。ありがとうございました!
guest

0

基本的に、そのチームのルールに従って操作すればいいです。

という模範的回答は横に置いとくことにして、
「GIT なかったことにする」でぐぐると、案外たくさんの解説があったりします。

投稿2018/06/06 14:20

y_waiwai

総合スコア87719

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

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

退会済みユーザー

退会済みユーザー

2018/06/06 14:31

ありがとうございます。 ぐぐるといろんな解決方法あってですね・・・普通どうやるのかなーなんて知りたかったわけです。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問