k9982874
V2EX  ›  问与答

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

  •  
  •   k9982874 · Feb 26, 2019 · 1833 views
    This topic created in 2659 days ago, the information mentioned may be changed or developed.

    例如

    <span >Hello world</span>
    

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

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

    人还是关注逻辑等
    littleMaple
        5
    littleMaple  
       Feb 26, 2019 via iPhone
    有一些自动化检测并强制 code style 的工具,可以了解一下
    lynskylate
        6
    lynskylate  
       Feb 26, 2019 via Android
    我注释单词间多了一个空格被 leader 指出来了...
    其实我觉得无所谓,只要你不是只关注产量命名,代码格式的问题,指出来我还是很乐意改的
    v2zhao
        7
    v2zhao  
       Feb 27, 2019 via Android
    @lynskylate 运气真好
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Solana   ·   1222 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 40ms · UTC 23:31 · PVG 07:31 · LAX 16:31 · JFK 19:31
    ♥ Do have faith in what you're doing.