masterに取り込まれた場合は修正ブランチをきれば良いでしょうか?
また、取り込まれる前はpushしなおせば良いのでしょうか?
気になる質問をクリップする
クリップした質問は、後からいつでもMYページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
回答2件
0
ベストアンサー
何のブランチをPushしたかにもよりますが、
master以外のブランチならセーフ、masterなら何やってんのという話になります。
プルリクエストのレビューや自動テストを使って、確実にmasterには動くと保証出来る状態を保ちましょう。
masterに取り込まれた場合は修正ブランチをきれば良いでしょうか?
1人プロジェクトならこっそりmasterを直で触って修正すれば良いと思いますし、
チームを組んで開発しているならごめんねを込めたプルリクエストを出すという正規の手順でmasterを修正しなおすべきでしょう。
バージョンを確定して後から追いやすくする目的で「タグ」機能を使いましょう。
また、取り込まれる前はpushしなおせば良いのでしょうか?
master以外のブランチAをpushし、プルリクエストをmasterに投げるとします。
この状態で、Aに更にコミット履歴を積み上げてpushすると、プルリクエストに今追加したコミット履歴を含めた状態のプルリクエストに進化します。
なのでmasterに取り込まれる前ならセーフ。
プルリクエストで何をやりたいのかというと、他人の目でレビューがしたいわけです。
- 実装者: こんなパッチ作ったよ取り込んでくれる?
- レビュワー: この行バグ出るんじゃね?不要なコメント見つけたから削除よろしく。この行コーディング規約無視してるぞ
- 実装者: ごめんごめん直しとくわ
- 実装者: パッチ修正してpushしたから再度確認お願いね
- レビュワー: 確かに指摘箇所全部直ってるな、よしマージ実行!
こういったやり取りがスムーズに行えるようになっています。
投稿2018/06/06 14:28
総合スコア21158
あなたの回答
tips
太字
斜体
打ち消し線
見出し
引用テキストの挿入
コードの挿入
リンクの挿入
リストの挿入
番号リストの挿入
表の挿入
水平線の挿入
プレビュー
質問の解決につながる回答をしましょう。 サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。 また、読む側のことを考えた、分かりやすい文章を心がけましょう。
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
退会済みユーザー
2018/06/06 14:30