- Git 12©2ch.net
469 ：デフォルトの名無しさん[sage]：2015/05/19(火) 11:07:38.39 ID:0hZF5IeJ - >>467
> このとき、2が1無しでも正しく動作するか分からないからね。
1+3のテストだけやればよくね？ 2がテストに通らなくても、
自動テストがあるならすぐにわかるし、最悪テストに通らないときだけやらなければいい。
もし自動テストがなければ、2がテストに通らなくても
「調べようがないので、通らないことがわからない=通っているのと一緒。」 ←重要
それにさ、コミットが多かったら、正しく動くことを担保するのは大変だよ。
* Aが正しく動くことを確認
* Bが正しく動くことを確認
* Cが正しく動くことを確認
* Dが正しく動くことを確認
というつもりだったが、たいていバグはあとから見つかるんで、
あとからAが正しく動いていなかった！と判明したらどうする？
* Aが正しく動かない
* Bが正しく動かない
* Cが正しく動かない
* Dが正しく動かない
* Eで修正
君は、どのコミットでも、正しく動くことを担保したいといったが
A〜Dは全部正しく動かないことが担保されたわけだｗ
EをAに混ぜ込めば直すことができるが、それをしないならば、A〜Dのコミットは正しく動かないままだよ。
A〜Dは正しく動かないが、でもその時はテストコードがなかったから分からないわけだよね。
それって、つまり「重要」って書いた所と同じ状態なんだが。
過去のコミットが動かないのは構わないというルールなら、そのまま放置でいいかもしれんが（笑）
|
- Git 12©2ch.net
470 ：デフォルトの名無しさん[sage]：2015/05/19(火) 12:47:36.82 ID:0hZF5IeJ - >>468
コミットのやり方？
普通にgit addでコミット対象を選んで
git commitでコミットするだけだけど？
コミットの順番を入れ替える話？
それならgit rebase -iだよ。
|
- 【JavaScript】ｽｸﾘﾌﾟﾄ ﾊﾞﾄﾙﾛﾜｲﾔﾙ49【php,py,pl,rb】 [転載禁止]©2ch.net
542 ：デフォルトの名無しさん[sage]：2015/05/19(火) 13:12:13.10 ID:0hZF5IeJ - >>541
既にあるものにちょこちょこっと機能追加するときには使えないよ。
wordpressの話をするならば、wordpressの管理画面を
全部作りなおす時に、使おうかどうか考慮するもの。
|
- Git 12©2ch.net
472 ：デフォルトの名無しさん[sage]：2015/05/19(火) 17:37:58.03 ID:0hZF5IeJ - >>471
バグの話ではこうなるよね？
つまりこれは「正しく動かないがテストコードがないから自動テストには通る」
という状態をどう考えてるかって話なんだよ。
俺はそれは自動テストに通るってことでいいじゃないかって言っている。
これがいやなら、前のコミットに混ぜて修正するしか無いわけだよ。
で、俺は 前のコミットに混ぜても自動テストに通るなら
それでいいじゃないかとも言ってる。
ちなみに俺は特に大きな理由がない限り、テストケースも含めて
前のコミットに混ぜて、全てのコミットを正しく動くようにしている。
皮肉なことに>>467が言ってる
> どっちのコミットでも正しく動作することを担保したい。
を実践出来てるのはこのやり方なんだ。
|
- Git 12©2ch.net
473 ：デフォルトの名無しさん[sage]：2015/05/19(火) 17:38:54.11 ID:0hZF5IeJ - ×テストケースも含めて
○（自動）テストコードも含めて
|