写代码不格式化的，都是垃圾，不接受反驳！

[helloworld1024]

[astrorobbie]

赞成

[461da73c]

C++ 的很难推动，本想推行 ci 自动格式化检查，一直都有人反对，很难推动下去。大家乐此不疲的 review 的时候 comment 格式化问题。

[jenlors]

持续集成加代码格式检测可解忧

[tulongtou]

赞成

[cmdOptionKana]

这个问题不大吧，自己自动格式化 1 秒的事。

[chendy]

赞成，但是老项目是真的不敢 format ，一个 format 下去杀伤范围过于大了……

[Rache1]

@cmdOptionKana 😂 主要是老项目，格式化的时候，一大片，review 简直要了老命。

[seers]

人生苦短，请用 gofmt

[461da73c]

@chendy 可以只格式化 patch 上下文的行。

[yuruizhe]

@Rache1 格式化又不会改变逻辑…除非 py2 到 py3 这种，老代码 print 没括号

[Rocketer]

ci 加格式检查只会查本次改动的文件，老文件不影响的

[chendy]

@461da73c idea 有这个功能，但是上下文识别总是出问题，每次都会 format 更大的范围，一不留神就把一大坨东西都给 format 掉了，只能把自己改的复制一下，revert 回去再粘贴上，后来就真的把 format 快捷键扣了不敢用了……

[Vegetable]

其实是 leader 的问题...

[a1562619919]

记一次坑。用 ide 格式化触发未知 bug ，这导致提交代码前编译正常，格式化后提交代码项目编译就挂了。。

[nightwitch]

@a1562619919
如果是 C++的话，代码或者注释里有中文的话老版本的 MSVC 会出现这种奇奇怪怪的 bug(GBK 编码问题)。
新版本的加 /utf-8 flag 就行。

[ALLROBOT]

@chendy 好奇怎么会会杀害力巨大呢？格式化不改变代码运行逻辑吧

[xiangyuecn]

不是所有代码都适合去做格式化🐶

[Rocketer]

说到底，格式化的问题都是历史问题，如果一直强制格式化，也就没那些问题了。

[Tyanboot]

@ALLROBOT 杀伤力巨大可能是指你一格式化，直接几千几万行的改动，到时候混在 diff 里面头都大了。

[rekulas]

@chendy 我们都是接手时专门提个点先把库格式化了再上手就没这问题了，不过要保证格式化不会出错而且所有人规则相同
以前在大公司工作格式化一行代码都要独立出来提 commit ，不然直接拒掉