Checklist for code reviews. Contribute to code-review-checklists/java-concurrency development by creating an account on GitHub.
Oct 13, 2023 bae5adc·Oct 13, 2023 History 17 Commits LICENSE Initial commit Sep 2, 2020 README MIT license Code Review Checklist Implementation Does this code change accomplish what it is supposed to do? Can this solution be simplified?
Has been peer review tested? Have edge cases been tested? Are invalid inputs validated? Are inputs sanitised? Documentation Is there sufficient documentation? Is the ReadMe.md file up to date? Other Has the release been annotated (GA etc)?
《Google Engineering Practices Documentation》:https://google.github.io/eng-practices/review/ 《Code Review at Cisco Systems》:https://static1.smartbear.co/support/media/resources/cc/book/code-review-cisco-case-study.pdf Wikipeida:https://en.wikipedia.org/wiki/Code_review...
checklist 可参考Code Review 速查手册 利用自动化工具进行前置检查 单元测试检查 新增单元测试检查 方法行数过多 圈复杂度过高 代码规范检查 lint 检查 体积监控 建议平均时长不要超过10分钟, 所以 e2e,性能检查等建议不阻塞发起MR流程 合理的规模 https://smartbear.com/learn/code-review/best-practices-for-peer...
Code Review(下文简称CR),即代码审查,是一种通过评审代码以发现并修正错误的实践。它不是一个新概念,但在软件开发中,它的重要性毋庸置疑。首先,它可以显著降低软件中的缺陷比例;其次,它促进了知识共享,通过评审的过程,团队成员可以相互学习,增强对系统的整体理解;最后,CR是一种预防措施,它有助于维护代码的清晰和...
推荐自己做一个 checklist 把自己当作 Reviewer 来对自己的代码进行 CR 预估代码可能出问题的地方 进行充分自测,保证代码可运行 不要指望别人帮你找出问题 checklist 可参考Code Review 速查手册 利用自动化工具进行前置检查 单元测试检查 新增单元测试检查
Code Review(下文简称CR),即代码审查,是一种通过评审代码以发现并修正错误的实践。它不是一个新概念,但在软件开发中,它的重要性毋庸置疑。首先,它可以显著降低软件中的缺陷比例;其次,它促进了知识共享,通过评审的过程,团队成员可以相互学习,增强对系统的整体理解;最后,CR是一种预防措施,它有助于维护代码的清晰和...
checklist 可参考Code Review 速查手册 利用自动化工具进行前置检查 单元测试检查 新增单元测试检查 方法行数过多 圈复杂度过高 代码规范检查 lint 检查 体积监控 建议平均时长不要超过10分钟, 所以 e2e,性能检查等建议不阻塞发起MR流程 合理的规模 https://smartbear.com/learn/code-review/best-practices-for-peer...
checklist 可参考Code Review 速查手册 利用自动化工具进行前置检查 单元测试检查 新增单元测试检查 方法行数过多 圈复杂度过高 代码规范检查 lint 检查 体积监控 建议平均时长不要超过10分钟, 所以 e2e,性能检查等建议不阻塞发起MR流程 合理的规模 https://smartbear.com/learn/code-review/best-practices-for-pe....