大家公司里前端提交和审查代码的流程是什么吖

V2EX = way to explore

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

现在注册

已注册用户请登录

V2EX 提问指南

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

小公司，之前只有三个前端，各管各的功能。现在另一个部门合过来了，领导让我定下代码管理的规范。。。

向各位大佬取经🥺

10 条回复 • 2019-12-15 14:10:06 +08:00

2019-12-15 12:09:40 +08:00 via iPhone

各管各的功能？你们没有前端离职过么？离职之后代码没人管么？

yuang

2019-12-15 12:10:25 +08:00 via Android

代码规范用 eslint 和 pritter，基本上不用自己定什么规范，定了也没人家现成的好

temporary

2019-12-15 12:13:56 +08:00

@wd #1 之前只有一个离职的。。各管各但都知道大家在干嘛
我离职了可能会有点影响也不知道今年涨多少工资

temporary

2019-12-15 12:15:55 +08:00

@yuang #2 会 Code Review 么
哎偶尔会有紧急的活那时候不知道 Code Review 该怎么操作

2019-12-15 12:17:05 +08:00 via iPhone

@temporary #3 我感觉你没明白我的意思。离职人的代码如果架构和逻辑一团糟，那比如你接手的时候怎么看？代码规范就是让大家写出来的代码任何人维护都不成问题，像自己写的一样。

temporary

2019-12-15 12:23:38 +08:00

@wd #5 是个问题。还没考虑过😅
之前那人离职，然后我接收理，他的代码架构虽然不是很舒服但也能理清楚。。。
得加上 Code Review 流程么
你们公司怎么操作的

jugelizi

2019-12-15 13:01:58 +08:00

工资够多就看看
不然谁有闲工夫审查别人的代码

zhuowenli

2019-12-15 13:39:32 +08:00 via Android

我们前端都开始推 DevOps 了 (心好累 😩

wellcheng

2019-12-15 13:48:23 +08:00

一开始是自建的 gitlab，后来是一套集成工具前端 + 后端 gitlab 支持

chenliangngng

2019-12-15 14:10:06 +08:00 via Android

最严格的 eslint，各管个的功能模块，开发阶段不审核测试阶段会要求从个人分支审核合入，不定期的 code review，离开的要留下交接文档说明全部负责的业务截图、代码和功能