V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
temporary
V2EX  ›  问与答

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

  •  
  •   temporary · 2019-12-15 11:39:38 +08:00 · 1999 次点击
    这是一个创建于 1565 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

    向各位大佬取经🥺

    10 条回复    2019-12-15 14:10:06 +08:00
    wd
        1
    wd  
       2019-12-15 12:09:40 +08:00 via iPhone
    各管各的功能?你们没有前端离职过么?离职之后代码没人管么?
    yuang
        2
    yuang  
       2019-12-15 12:10:25 +08:00 via Android
    代码规范用 eslint 和 pritter,基本上不用自己定什么规范,定了也没人家现成的好
    temporary
        3
    temporary  
    OP
       2019-12-15 12:13:56 +08:00
    @wd #1 之前只有一个离职的。。各管各但都知道大家在干嘛
    我离职了可能会有点影响 也不知道今年涨多少工资
    temporary
        4
    temporary  
    OP
       2019-12-15 12:15:55 +08:00
    @yuang #2 会 Code Review 么
    哎偶尔会有紧急的活 那时候不知道 Code Review 该怎么操作
    wd
        5
    wd  
       2019-12-15 12:17:05 +08:00 via iPhone
    @temporary #3 我感觉你没明白我的意思。离职人的代码如果架构和逻辑一团糟,那比如你接手的时候怎么看?代码规范就是让大家写出来的代码任何人维护都不成问题,像自己写的一样。
    temporary
        6
    temporary  
    OP
       2019-12-15 12:23:38 +08:00
    @wd #5 是个问题。还没考虑过😅
    之前那人离职,然后我接收理,他的代码架构虽然不是很舒服但也能理清楚。。。
    得加上 Code Review 流程么
    你们公司怎么操作的
    jugelizi
        7
    jugelizi  
       2019-12-15 13:01:58 +08:00
    工资够多就看看
    不然谁有闲工夫审查别人的代码
    zhuowenli
        8
    zhuowenli  
       2019-12-15 13:39:32 +08:00 via Android
    我们前端都开始推 DevOps 了 (心好累 😩
    wellcheng
        9
    wellcheng  
       2019-12-15 13:48:23 +08:00
    一开始是自建的 gitlab,后来是一套集成工具前端 + 后端 gitlab 支持
    chenliangngng
        10
    chenliangngng  
       2019-12-15 14:10:06 +08:00 via Android
    最严格的 eslint,各管个的功能模块,开发阶段不审核测试阶段会要求从个人分支审核合入,不定期的 code review,离开的要留下交接文档说明全部负责的业务截图、代码和功能
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   5255 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 26ms · UTC 09:36 · PVG 17:36 · LAX 02:36 · JFK 05:36
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.