例如
<span >Hello world</span>
我会指出来但是感觉有点龟毛
1
66450146 2019-02-26 16:39:01 +08:00 via iPhone
这种代码一般不用等到 review,自动化工具就要亮红灯了
|
2
morefreeze 2019-02-26 18:20:45 +08:00
你可以指出你们 CI 没有集成 lint 工具
|
3
hilbertz 2019-02-26 18:22:36 +08:00
空格又不违反基.本法
|
4
9hills 2019-02-26 18:36:47 +08:00 via iPhone
集成 lint 啊,不通过不让合入就完了
人还是关注逻辑等 |
5
littleMaple 2019-02-26 18:42:28 +08:00 via iPhone
有一些自动化检测并强制 code style 的工具,可以了解一下
|
6
lynskylate 2019-02-26 18:46:38 +08:00 via Android
我注释单词间多了一个空格被 leader 指出来了...
其实我觉得无所谓,只要你不是只关注产量命名,代码格式的问题,指出来我还是很乐意改的 |
7
bunnyblueair 2019-02-27 09:15:32 +08:00 via Android
@lynskylate 运气真好
|