temporary
V2EX  ›  问与答

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

  •  
  •   temporary · Dec 15, 2019 · 2683 views
    This topic created in 2346 days ago, the information mentioned may be changed or developed.

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

    向各位大佬取经🥺

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