在提交代码前,我经常先做 git add 来添加相关文件、目录,然后运行 git diff --staged 来检查我所做的尚未commit的更改。通常我检查这些事项: 我是否留了个注释或者TODO? 变量名是否合适? 以及其它我上面提到的 我想确认我先通过我自己的代码审查然后再呈献到其他人面前。这通常会使你少受刺激--自己批评自己好过...
I want to make sure that I would pass my own code review first before I subject other people to it. It also stings less to get notes from yourself than from others :p 在提交代码前,我经常先做 git add 来添加相关文件、目录,然后运行 git diff --staged 来检查我所做的尚未commit的更改。通...
In the meanwhile, people who "aren't" programmers but are more of a mathematician, physicist, algorithm developer, scientist, you name it commit sins mostly of the following kinds: Long functions Bad names (m, k, longWindedNameThatYouCantReallyReadBTWProgrammersDoThatALotToo) Access all over t...
to create a new review from a single commit, or to create areview that tracks a whole branch– this last option will automatically add all new commits on this branch to this review.
It is a lightweight peer code review tool that is used in pre-commit and post-commit reviews. It has capabilities for SVN, Perforce and CVS. Review Board A web-based, collaborative, free and open source tool used for used by source team projects and companies. ...
https://google.github.io/eng-practices/review/developer/small-cls.html Commit 描述 Bad Case: “修复错误”是不充分的 CL 描述。什么 bug?你做了什么来修复它? 其他类似的不良描述包括: 逻辑修复 添加补丁 增加XX规则 删除XX逻辑 Good Case: ◆ 摘要:【xx模块】新增xx功能 ...
If you aren’t ready, send the author any thoughts or questions you have so far, and schedule and commit to a time when you intend to do a more detailed pass and approve or request changes. Keep two separate repositories on your machine, one for your own changes and one for changes ...
Sairyss/backend-best-practicesPublic NotificationsYou must be signed in to change notification settings Fork122 Star1.6k Code Issues master 2Branches 0Tags Code Latest commit Sairyss updated 'Logging' section Aug 30, 2024 f87fcd7·Aug 30, 2024 ...
You can right-click on the file and show changes, revert changes, and update or commit. Using the VisualSVN menu, you can manage changes for the entire project. The VisualSVN menu also contains shortcuts to common TortoiseSVN functions like the repo-browser, creating and applying patches, ...
Our review tasks usually are non-blocking, so developers are free to commit their changes (we do post-commit reviews for small stories) and then ask another developer to review their codes while they continue with their other user stories. Generally, we do not get many tasks waiting for revi...