- Git 12©2ch.net
35 ：デフォルトの名無しさん[sage]：2015/03/25(水) 13:40:29.78 ID:1QZDV4EM - >>33
レビューっていうのは、プルリクエストが出された「ひとまとまりのコミット」に対して行われるものだ
その時にレビュアーは、その「ひとまとまりのコミット」が何のための修正かを把握しているはずである
※1つのプルリクエストに複数の目的の修正が入っているなら、そのプルリクエストが間違ってる
レビュアーが、その「ひとまとまりのコミット」が何のためなのかを分かっていれば、各コミットが
どれだけ細分化されていようと、（レビュアーは「ひとまとまりのコミット」で最終的にどうなったかだけを
チェックし、各コミットで何をしたかまでは追いかけない、という理由で）コードが合っているかの
チェックを阻害したりはしない
これで反論になってるだろうか
|
- Git 12©2ch.net
44 ：デフォルトの名無しさん[sage]：2015/03/25(水) 20:00:32.67 ID:1QZDV4EM - >>43
（33で言ってた内容）
＞途中のコミットの話が抜けてる。同じ内容になるからという理由だけなら
＞途中のコミットが汚なくてもいい（レビューが困難）でもいいって話になってしまう。
＞rebaseの目的はレビューを容易にすること。
（43で言ってた内容）
＞いきなり複数のファイルにまたがる1000行の修正を送られたって
＞何をしたいのかわからない。
プルリクエスト1件を送るとき、コミットをまとめるべきと言ってたはずが、
コミットをまとめないべき、にすりかわっている。
あなたの意見どころかあなた自身が、反論に耐えうるものじゃないんじゃなかろうか
|
- Git 12©2ch.net
47 ：デフォルトの名無しさん[sage]：2015/03/25(水) 20:17:13.12 ID:1QZDV4EM - >>45
あなた自体が説明に耐えうるものじゃないな・・・
自分の主張や情報を後出し後出しにして、ハイ反論どうぞ、なんて言われてもねぇ
そんなんじゃ、周りの人とまともな議論ができないだろう・・・
自分が推測するに、あなたは下請けに仕事を発注したことはあるが、
「チームで開発作業をしたことはあまりない」と感じられるんだが、実際どうだろうか？
まぁおそらくは、反論してくるだろうと思うが
|
- Git 12©2ch.net
53 ：デフォルトの名無しさん[sage]：2015/03/25(水) 21:18:09.55 ID:1QZDV4EM - >>49
>>48に書いたことを、>>33のときに書いていれば反論を招かずに済んだ話だと思う
自分が言いたいのは、まさにこの一点
このスレをgitに例えれば、あなたは>>33におけるタイプミスの修正を>>51まで延々と
続けているということだ
|
- Git 12©2ch.net
57 ：デフォルトの名無しさん[sage]：2015/03/25(水) 21:32:07.66 ID:1QZDV4EM - >>54
＞お前の人間として品性がないって話をしている。
違うだろう、今度はいつの間に「反論があるかないか」の話からすりかわったんだ
まぁでもお前がgitのスレでやりたいのは、反論があるかないかじゃなく、人を馬鹿にする方なんだろうが・・・
|
- Git 12©2ch.net
59 ：デフォルトの名無しさん[sage]：2015/03/25(水) 21:46:25.46 ID:1QZDV4EM - 自分には、どうして>>47の後半で言ったことがお前をそこまで刺激したのかが、よく分からないんだよ
けっきょく、このスレで言った言わないの議論をするお前じゃなくて、今技術者をやってるお前ってどうなの？
なぜ軽くスルーできなかったのか？やっぱ、そこに問題があるから怒ったのか？そうだったら謝りたいが
|
- Git 12©2ch.net
61 ：デフォルトの名無しさん[sage]：2015/03/25(水) 21:53:44.04 ID:1QZDV4EM - >>60
第三者的には、>>58の時点でバカ2名のやり取りにしか見えないと思うぞ
|
- Git 12©2ch.net
63 ：デフォルトの名無しさん[sage]：2015/03/25(水) 22:23:48.78 ID:1QZDV4EM - >>62
そうかもね、すまない
|