服了，一个同事写的 demo，每次他 commit 后，我不经过曲折的 demo 代码修改，肯定是跑不通的。
northisland · 2018 年 3 月 13 日 · 3424 次点击这是一个创建于 2870 天前的主题，其中的信息可能已经有所发展或是发生改变。
哎，我都无语了，
天知道他是怎么确定这次 commit 是正确的。。。
我该怎么办
 |
1
kosilence 2018 年 3 月 13 日
Need code review.
|
|
2
kosilence 2018 年 3 月 13 日  2
And precommit unit test.
|
 |
3
fanqianger 2018 年 3 月 13 日
你需要单元测试吧，一般提交 review 后都会自动跑单元测试，如果没有通过，是不会有人 review 你的代码的
|
 |
4
northisland OP 想太多了，日常开发，公用一个 SVN （我也觉得 git 好，只是公司用的是这个）
|
 |
5
scnace 2018 年 3 月 13 日 via Android 1
svn 好像也可以 hook
|
 |
6
zjsxwc 2018 年 3 月 13 日 via Android
连 git 都不愿意切换的项目，让他们自己玩去吧
|
 |
7
shynome 2018 年 3 月 13 日 via Android 1
永远不动别人的代码。
你改好了，他又会改回去的，得一起找出问题才行 |
 |
8
3a3Mp112 2018 年 3 月 13 日 1
没人 review 的代码是怎么 commit 成功的？
|
|
9
northisland OP |
 |
10
HanSonJ 2018 年 3 月 13 日 4
@northisland #9 很多公司
|
 |
11
des 2018 年 3 月 13 日 via Android
@northisland 在自己分支上开发啊，不用每个 commit 都合并啊
|
 |
12
0x4b0082 2018 年 3 月 13 日
<del>很正常阿 谁规定只有写好的代码能提交上去的？</del>
emm 看了眼评论 svn 抱歉打扰了 |
 |
13
zachguo 2018 年 3 月 14 日 1
测试里加一项：验证 demo 能跑才能 merge，不然过不了 CI 打回去。
|
Select Language