[過去ﾛｸﾞ] Git 17 (1002ﾚｽ)
上下前次1-新
抽出解除 必死ﾁｪｯｶｰ(本家) (べ) 自ID ﾚｽ栞 あぼーん
このｽﾚｯﾄﾞは過去ﾛｸﾞ倉庫に格納されています｡
次ｽﾚ検索 歴削→次ｽﾚ 栞削→次ｽﾚ 過去ﾛｸﾞﾒﾆｭｰ
645(1): デフォルトの名無しさん [sage] 2021/04/24(土) 07:12:17.73 ID:lum8vFBO(1/4) AAS
> コミットの単位が適切であれば…苦労することはない、の部分が理由としては弱い気がするので。

「コミットの単位が適切であれば苦労することはない」は理由じゃなくて事実

逆マージってログを見ないでしょ？
残っていればOK。あとから頑張れるはず（希望。実際には頑張ったことはない）
見ないログはなんの役にも立たないし汚いログも見れないのでこれも役に立たない

>feature実装と解消を分けて考えられるので、
分けて考えてる時点でダメじゃん
リベースしてしまえば、実装だけしか考えなくて良くなる

バグが見つかった時、実装と解消の２つがあれば
実装のバグかも知れないし解消のバグかも知れない。
バグの可能性が２つに増えてしまっている
リベースしてしまえばそれがなくなる

> あ、適切という言葉に客観性がほしいという意味です。
いきあたりばったりで作業するなということ
どうせ、こう修正した、気分が変わってやり直した、レビューの指摘があったから変えた
バグが見つかって対応した。って同じ箇所を何度も修正したのがコミットに残ってるんでしょ？

このコミットをあとから見て役に立つとでも思ってんの？
一連のブランチである箇所のコードを修正しているコミットを一つに保つように
随時リベースしていれば何も困らない。masterが更新されるたびにリベースしていれば
何も困らない。コンフリクトがほぼ発生しない。それが適切ということ
649: デフォルトの名無しさん [sage] 2021/04/24(土) 19:46:18.78 ID:lum8vFBO(2/4) AAS
> >>646
646(3): デフォルトの名無しさん [sage] 2021/04/24(土) 10:18:40.57 ID:vKvwVCfN(1/2) AAS
コンフリクト解消のコミットでバグを作り込みやすいと考えるのか、それが重視されるほどのリスクではないと考えるのかで選択が変わるんじゃないかな
順不同で両方取り込めば済むようなほんの小さなコンフリクトであればrebaseの利点のみが優勢になるし、複雑なコンフリクトであれば逆マージの利点が優勢になる
複雑な解消作業までrebaseで混ぜてしまうともったいないし潔癖が過ぎる感じ
その辺をガイドラインにしつつ線引は各人のこだわりに任せるのがうまく回りそう
の仰るとおり複雑なコンフリクトであれば逆マージにしてそれ以外はリベースすると言うのが個人的な結論になりそうです

複雑なコンフリクト＝開発に失敗したブランチ
開発に失敗したから逆マージするしか手がなくなってる
仕事だと期日まで仕上げろと言われて、みんな諦めるが
これがきっちりとしたオープンソースの開発だと、そんなものマージできませんでリジェクトされる
650: デフォルトの名無しさん [sage] 2021/04/24(土) 19:50:06.67 ID:lum8vFBO(3/4) AAS
>>646
> コンフリクト解消のコミットでバグを作り込みやすいと考えるのか、

「コンフリクトの解消でバグが発生」という事自体がナンセンスだよね
だって開発自体は問題ないって認識なんでしょ？

開発自体に問題ないのに、コンフリクトでバグが発生するというのなら
それはもはや、コンフリクト解消という名の追加開発じゃんｗ

開発終わってるのに開発続けてどうするの
マージが問題なく完了できるという所までが開発でしょ？
651: デフォルトの名無しさん [sage] 2021/04/24(土) 19:59:19.02 ID:lum8vFBO(4/4) AAS
>>647
647(1): デフォルトの名無しさん [sage] 2021/04/24(土) 10:31:26.53 ID:Q6OLD2jb(1) AAS
>>642のおっしゃる通り友人の言うことがよく理解できてなかったんですが>>645でおおよそしっくり来ました
その上でもやはり、あるfeatureリリースによって別の機能がデグっていることが判明した場合などにおいて、逆マージコミットが残っていた方がよいのではないかと思いました
コンフリクト解消にてうっかり失敗したのか、新機能実装において意図を持って変えたのかが明確になるからです
>>646の仰るとおり複雑なコンフリクトであれば逆マージにしてそれ以外はリベースすると言うのが個人的な結論になりそうです
> その上でもやはり、あるfeatureリリースによって別の機能がデグっていることが判明した場合などにおいて、逆マージコミットが残っていた方がよいのではないかと思いました
> コンフリクト解消にてうっかり失敗したのか、新機能実装において意図を持って変えたのかが明確になるからです

いやさ、なんでコンフリクトが発生するのか理解してないの？
masterとブランチの両方でコードを修正したときなんだが

masterの修正は他の人がやってる。
コンフリクトが発生するってことは、他の人がやった開発内容を把握してないということ

コンフリクト解消時に他の人がやった開発内容を把握するなんて大変な作業をやるんか？
同じ箇所を修正してるのに、お前のコードはその他人の開発内容を踏まえた修正になってないだろ
ちゃんとテストしてるんか？してるわけないよなコンフリクト解消時に判明するぐらいなんだから

コンフリクト解消時に、ついでに一緒にって追加開発するなよ
近い場所を修正したときなど、簡単な修正が必要なコンフリクトは発生することはあるが
あとからコンフリクト解消の内容を精査するなんてことがあること自体がおかしい
上下前次1-新書関写板覧索設栞歴
ｽﾚ情報 赤ﾚｽ抽出 画像ﾚｽ抽出 歴の未読ｽﾚ AAｻﾑﾈｲﾙ
ぬこの手 ぬこTOP 0.043s