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

2018-03-13 19:41:54 +08:00
 northisland

哎,我都无语了,

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

我该怎么办

3039 次点击
所在节点    程序员
13 条回复
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
svn 好像也可以 hook
zjsxwc
2018-03-13 19:56:26 +08:00
连 git 都不愿意切换的项目,让他们自己玩去吧
shynome
2018-03-13 19:59:14 +08:00
永远不动别人的代码。
你改好了,他又会改回去的,得一起找出问题才行
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
@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 打回去。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/437732

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX