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

在 Code Review 时会指出空格问题么?

  •  
  •   k9982874 · 2019-02-26 16:35:33 +08:00 · 1376 次点击
    这是一个创建于 2132 天前的主题,其中的信息可能已经有所发展或是发生改变。

    例如

    <span >Hello world</span>
    

    我会指出来但是感觉有点龟毛

    7 条回复    2019-02-27 09:15:32 +08:00
    66450146
        1
    66450146  
       2019-02-26 16:39:01 +08:00 via iPhone
    这种代码一般不用等到 review,自动化工具就要亮红灯了
    morefreeze
        2
    morefreeze  
       2019-02-26 18:20:45 +08:00
    你可以指出你们 CI 没有集成 lint 工具
    hilbertz
        3
    hilbertz  
       2019-02-26 18:22:36 +08:00
    空格又不违反基.本法
    9hills
        4
    9hills  
       2019-02-26 18:36:47 +08:00 via iPhone
    集成 lint 啊,不通过不让合入就完了

    人还是关注逻辑等
    littleMaple
        5
    littleMaple  
       2019-02-26 18:42:28 +08:00 via iPhone
    有一些自动化检测并强制 code style 的工具,可以了解一下
    lynskylate
        6
    lynskylate  
       2019-02-26 18:46:38 +08:00 via Android
    我注释单词间多了一个空格被 leader 指出来了...
    其实我觉得无所谓,只要你不是只关注产量命名,代码格式的问题,指出来我还是很乐意改的
    bunnyblueair
        7
    bunnyblueair  
       2019-02-27 09:15:32 +08:00 via Android
    @lynskylate 运气真好
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2575 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 23ms · UTC 11:15 · PVG 19:15 · LAX 03:15 · JFK 06:15
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.