没想到公司新来小白竟然做了这样的事……

我已经疯了,今天公司新来的小白提交了好了问题代码到项目仓库,最要命的是项目在线上跑不起来时费劲好大力气,检查了各种可能性,甚至还一度怀疑是不是数据库、应用服务器有问题呢。花了半个小时才发现是代码问题,追踪到版本管理才发现是新来的小白提交了很多问题代码!!!

然后又花了很多时间为了把这个被小白污染的仓库清理干净,因为在他提交的前前后后都会其他的代码更新。

都怪我,为了着急上线没怎么做测试就匆匆上线;都怪我,没有检查大家提交的代码就急忙更新。业务中断两个小时,看来离滚蛋不远了。吃键盘的心都有了!!!

----- 华丽分割线 -----

这就是不做代码评审的下场!!!

可是代码评审很难搞,特别是使用 SVN 或者把 Git 当做 SVN 来用的团队。

我们希望项目的主仓库只有负责人才有权限操作,项目组成员对代码的任何修改都要经过审核后才能合并到主仓库。这样才能确保上述事故不会发生。

----- 呼啦啦分割线 -----

其实只要我们善用 Git 工作流就可以轻轻松松实现代码的评审功能。

以码云为例,基本流程如下:

  1. 假设项目主仓库 A/Project1 ,该仓库只有项目负责人(小红)具备写权限,项目成员只读
  2. 新来的小白复制 A/Project1 仓库到 "小白/Project1" (在 A/Project1 页面点一下 Fork 按钮)
  1. 然后小白开始制造各种 Bug ,修改无数代码
  2. 写了一天的代码,小白觉得可以交差了,然后创建了个 Pull Request 请求把当前的改动合并到主仓库
  1. 项目负责人小红看到有人提交了 Pull Request ,打开小白修改的文件一看,惊呼:这是什么垃圾代码!!!
  1. 一时间,小白哭,小红怒,天色阴暗,可能要下雨了。。。

如果你还不熟悉 Git 工作流的操作,请前往 https://gitee.com/ 体验。

欲知后事如何,请听下回分解。

原文发布于微信公众号 - 码云Gitee(mayunOSC)

原文发表时间:2018-05-21

本文参与腾讯云自媒体分享计划,欢迎正在阅读的你也加入,一起分享。

发表于

我来说两句

0 条评论
登录 后参与评论

相关文章

来自专栏Guangdong Qi

iOS开发常用之网络、网页

2571
来自专栏CRPER折腾记

VS Code 折腾记 - (11) 再来一波插件推荐!(代码片段,框架,Node,touchbar,TS,Git,数据库,python!!)

之前也多多少少写过几篇推荐插件的文章; 因为插件这类东西不是大风刮来的.所以我从最初的半个月推荐, 到现在慢慢接近三个月推荐一次,以后可能会更久也不一定!

2063
来自专栏知晓程序

我,一个自诩牛逼上天的 Node.js 和小程序开发者,今天就教「快应用」好好做人

1442
来自专栏阮一峰的网络日志

Usenet:P2P下载的替代方法

在上一篇网志中,我已经说了,我打算好好研究和介绍usenet。 下面就是我翻译的第一篇教程。它的信息量很大,非常通俗实用。基本上,读完这篇文章,你就会使用use...

3848
来自专栏冰霜之地

Vue 全家桶 + Electron 开发的一个跨三端的应用

我是一名全职的 iOS 开发者,非前端开发者。由于接触了 Weex 开发,从而接触到了 Vue.js。

3397
来自专栏app=

省油一号分销系统模式开发源码

在平时开发程序的过程中,自己经常会写一些控制台进行测试某个功能,事后我们会寻找这些程序,如果不仔细管理,经常会找不到。由于每个控制台程序都有自己独立的 Main...

3317
来自专栏逸鹏说道

漫谈前端优化

近几年,移动端站点和app、手游井喷式的发展,大大出乎了当初从业者的预料,2010年前后中国还处于3G网时代,各种3G宣传铺天盖地,如果你稍稍回忆一下,那个时候...

3649
来自专栏美丽应用

玩转Volumio的DLNA特性

3513
来自专栏CRPER折腾记

MacOS : 前端必备姿势(工作环境)

算是爬出来了...此篇的工作设备是(Macbook Pro 2017款,所以可能会说到 touchbar!)

4760
来自专栏鹅厂优文

开发效率太低?您可能没看这篇文章

还记得刚参加工作的时候, 有位开发的同事软件使用效率奇高. 我曾亲眼目睹他在几秒之内打开开发软件, 优雅地调出隐藏的功能, 输入数据输出结果的过程行云流水, 一...

7072

扫码关注云+社区

领取腾讯云代金券