代码评审的最小可用配置：五个必须改的默认值

评审等待超过一天，上下文切换的成本会让作者宁愿推倒重写

下面这份配置的目标不是最优，是「半年后回来看还认得」。先把这五项改掉。

先说结论

最小可用配置 = 能启动 + 能观测 + 能备份 + 能回滚。缺一项都不算配好。

• 路径与数据目录先定死，别用默认
• 日志要能看，且要有轮转
• 资源上限显式写出来
• 健康检查与重启策略配好
• 备份 + 回滚演练一次

五项必改

按顺序改，每改一项重启验证一次：

• 数据目录：改到独立分区或卷，避免日志写满根分区
• 监听地址与端口：只监听需要的接口，别 0.0.0.0 全开
• 资源上限：内存、CPU、句柄数显式限制
• 日志：级别、路径、轮转策略三样都写
• 时区：容器与宿主一致，否则日志和定时任务全乱

配置长什么样

评审等待超过一天，上下文切换的成本会让作者宁愿推倒重写。

表里的值不是标准答案，是「先按这个来，再按观测调」。

这个服务的几个关键事实

和默认值相关、但文档里往往一笔带过的几条：

• 评审等待超过一天，上下文切换的成本会让作者宁愿推倒重写
• 把格式问题交给 lint 后，人工评论里讲空格的占比能从三成降到 5% 以下
• 评论要区分「必须改」和「可以讨论」，否则作者无法判断优先级
• 同一个 PR 混入重构和功能改动，diff 会放大 2-3 倍

验证：怎么知道配对了

配完必须验证，不能只看进程在：

• 重启一次，确认能自动起来
• 故意打满内存或断开依赖，看是否按预期降级
• 查看日志，确认时间戳与本机一致
• 跑一次备份，再从备份恢复到一个空目录

配置怎么管

把配置放进版本控制，改动写注释说明原因。半年后你会感谢现在的自己。

还有几处容易忽略的

都是小地方，但出问题时往往就卡在这里。

• 单次评审超过 400 行时缺陷发现率明显下降，超过 1000 行基本只剩风格意见
• 评审记录是团队知识的载体，三个月后翻记录比翻文档有效
• PR 攒了一周才提，两千行 diff 没人看得下去
• 只评论「这里可以更好」不说为什么，作者改不动
• 在 PR 里顺手重构无关文件，真正的改动被淹没

默认值能用，但只有改过的配置才叫「你的」。