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

2019-02-26 16:35:33 +08:00
 k9982874

例如

<span >Hello world</span>

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

1360 次点击
所在节点    问与答
7 条回复
66450146
2019-02-26 16:39:01 +08:00
这种代码一般不用等到 review,自动化工具就要亮红灯了
morefreeze
2019-02-26 18:20:45 +08:00
你可以指出你们 CI 没有集成 lint 工具
hilbertz
2019-02-26 18:22:36 +08:00
空格又不违反基.本法
9hills
2019-02-26 18:36:47 +08:00
集成 lint 啊,不通过不让合入就完了

人还是关注逻辑等
littleMaple
2019-02-26 18:42:28 +08:00
有一些自动化检测并强制 code style 的工具,可以了解一下
lynskylate
2019-02-26 18:46:38 +08:00
我注释单词间多了一个空格被 leader 指出来了...
其实我觉得无所谓,只要你不是只关注产量命名,代码格式的问题,指出来我还是很乐意改的
bunnyblueair
2019-02-27 09:15:32 +08:00
@lynskylate 运气真好

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/538933

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX