cinatra icon indicating copy to clipboard operation
cinatra copied to clipboard

建议增加clang-format配置文件用以统一代码风格

Open Jackson-soft opened this issue 5 years ago • 8 comments

Jackson-soft avatar Mar 07 '19 02:03 Jackson-soft

小改动请 @Jackson-soft 代劳下?

micfan avatar Mar 16 '19 10:03 micfan

我觉得没必要专门用clang搞,规定2个或4个空格就好。

qicosmos avatar Mar 17 '19 01:03 qicosmos

我认为这是一个开源项目最基本的配置。

jovany-wang avatar Mar 17 '19 02:03 jovany-wang

这要求使用者编译的时候还需要装clang,如果人家没装clang呢?

qicosmos avatar Mar 17 '19 02:03 qicosmos

很多开源项目也没有专门搞个clang去format啊,何谈最基本的配置。

qicosmos avatar Mar 17 '19 03:03 qicosmos

我所知道的项目,不管是公司的,还是开源的,没有不配check style的。

  1. clang 和clang-format是没有啥关系的, clang-format只是个代码格式化工具,并不要求装clang。
  2. 即使clang-format也不要求开发者装,而是ci中enable即可。

建议多参考知名的开源项目。

jovany-wang avatar Mar 17 '19 03:03 jovany-wang

现在都没有和ci集成,你可以给出一个方法,要求是让用户clone下来就能直接用这个项目,同时也能格式化好代码。 我觉得也许可以在cmakelist里面加一个编译选项,默认关闭,如果需要格式化就打开开关。

qicosmos avatar Mar 17 '19 03:03 qicosmos

另外我不觉得你参与了万颗star的项目和我们讨论的问题有什么关系,我更期望有实际的方法,你甚至可以提pr,欢迎参与。

qicosmos avatar Mar 17 '19 03:03 qicosmos

done

qicosmos avatar Jan 14 '24 12:01 qicosmos