一一中文网

手机浏览器扫描二维码访问

第45章 代码审查（第3页）

。

陈峰翻到最后，“你用了内联样式和大量!important覆盖。

我们项目用tailwindcSS，要求原子化样式类。

另外，响应式设计考虑不足，在移动端布局会乱”

。

他停下来，看向林晨：“暂时就这些主要问题。

细节评论我都在代码行里加了，总共……四十七条评论”

。

四十七条。

林晨感觉脸上有些发烫。

工作十年，他从未在一个pR里收到过这么多修改意见。

“陈哥，抱歉，是我没熟悉好规范”

。

他诚恳地说。

陈峰摆摆手：“不用道歉，新人第一个pR都这样。

web3开发，尤其是金融相关，安全性和代码质量的要求比传统web高几个数量级。

一笔错误交易可能造成用户永久资产损失，一个合约漏洞可能被黑客利用导致项目归零。

所以审查严格是必须的”

。

他顿了顿，语气缓和了些：“但你的优势也很明显——实现速度快，基础逻辑扎实，没有低级错误。

只是需要适应我们的标准和思维方式”

。

林晨点头：“我回去一条条改。

大概需要多久”

？

“给你两天时间”

。

陈峰说，“改完后重新提交，我会再审查。

另外，建议你仔细读一下项目里的coNtRIbUtING.md贡献指南，还有code-review-checklist.md代码审查清单，里面写了所有要求和常见问题”

。

“好的，我马上看”

。

回到工位，林晨先点开陈峰说的那两个文档。

贡献指南足足有三十多页，从Git工作流、提交信息规范、代码风格、测试要求，到安全注意事项、性能优化点，事无巨细。

代码审查清单更是列了一百多项检查项。

他泡了杯浓茶，开始逐条处理评论。