服了，一个同事写的 demo，每次他 commit 后，我不经过曲折的 demo 代码修改，肯定是跑不通的。

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 2439 天前的主题，其中的信息可能已经有所发展或是发生改变。

哎，我都无语了，

天知道他是怎么确定这次 commit 是正确的。。。

我该怎么办

commit

Demo

无语

不通

13 条回复 • 2018-03-14 10:20:16 +08:00

kosilence

2018-03-13 19:46:21 +08:00

Need code review.

kosilence

2018-03-13 19:46:48 +08:00

And precommit unit test.

fanqianger

2018-03-13 19:48:26 +08:00

你需要单元测试吧，一般提交 review 后都会自动跑单元测试，如果没有通过，是不会有人 review 你的代码的

northisland

2018-03-13 19:51:50 +08:00

想太多了，日常开发，公用一个 SVN （我也觉得 git 好，只是公司用的是这个）

scnace

2018-03-13 19:52:37 +08:00 via Android

svn 好像也可以 hook

zjsxwc

2018-03-13 19:56:26 +08:00 via Android

连 git 都不愿意切换的项目，让他们自己玩去吧

shynome

2018-03-13 19:59:14 +08:00 via Android

永远不动别人的代码。
你改好了，他又会改回去的，得一起找出问题才行

3a3Mp112

2018-03-13 20:00:02 +08:00

没人 review 的代码是怎么 commit 成功的？

northisland

2018-03-13 20:10:26 +08:00

@3a3Mp112 我不是他上司，也不是他导师……

我们出大版本，是要 review，jekins 的，

哪个公司每次 commit 都要 review ？

HanSonJ

2018-03-13 20:11:38 +08:00

@northisland #9 很多公司

des

2018-03-13 20:19:04 +08:00 via Android

@northisland 在自己分支上开发啊，不用每个 commit 都合并啊

raphaelsoul

2018-03-13 20:22:37 +08:00

<del>很正常阿谁规定只有写好的代码能提交上去的？</del>

emm 看了眼评论 svn 抱歉打扰了

zachguo

2018-03-14 10:20:16 +08:00

测试里加一项：验证 demo 能跑才能 merge，不然过不了 CI 打回去。