今日推荐开源项目：《和某海王星没关系 reborn》
今日推荐英文原文：《Better Code Reviews》
今日推荐开源项目：《和某海王星没关系 reborn》传送门：GitHub链接
推荐理由：这个作者的仓库里有不少很有意思的书，这个就是其中之一，通过讲述各式各样的在现实生活中并不少见的现象（有时例子就是他自己）来说明一些容易理解的道理。有些道理实际上并不是太复杂，日常生活中每一天都能看见都在无意识的被他影响，这本书能让读者注意到这些没有被重视或者说直接被无视的点。
今日推荐英文原文：《Better Code Reviews》作者：Alex Persian
原文链接：https://medium.com/better-programming/better-code-reviews-190efd53bc10
推荐理由：更好的处理 Pull Request 的方式
Better Code Reviews
Become a more effective and empathetic reviewer
Reviewing code is an daily occurrence for developers. It can be a humbling learning experience, but it can also turn into an egocentric process. Over the last few years I’ve learned some best practices, either through personal experience or excellent advice from peers, that have helped me to become a more effective and empathetic reviewer. I wanted to share some of this information, in the hopes that it will help others. This short post is broken into two main parts: 1. getting the best feedback on your work and 2. providing great reviews for your peers.Structuring Your PR’s to Make Their Life Easy
@iamdeveloper https://twitter.com/iamdevloper/status/397664295875805184
Avoid massive changelogs
- Keep it small — under 300 lines of changes, if possible.
- If a piece of work requires more than 600 lines try to break it into smaller logical chunks that can be reviewed individually.
- No one wants to be sent a 1000 line PR monstrosity. Good reviewers will often reject a pull request of this scale; they know from experience that they probably won’t be able to to review it effectively.
Structure for comprehension
- Group similar changes into commits. Changing the location of a bunch of files without modifying them? Group that into its own commit, so the reviewer knows it’s not a logical change they need to focus on.
- Commit changes frequently to reduce their scope. For larger pieces of work it is far easier for your reviewer to focus on what’s important when the commits are small.
Short-circuit possible confusion
- If you modify logic or structure that existed in a previous set of commits, ensure you inform the reviewer so they don’t get sidetracked into critiquing now non-existent code. This is applicable for when the reviewer is going through commit by commit.
Provide enough context
- Ensure that your PR is described clearly, whether this is through a linked ticket or a written description in the PR, or preferably both. Provide examples, visual or otherwise, of how your work affects the project. Look at things from the reviewer’s point of view and try to make it as seamless as possible for them to understand the purpose of your pull request.
Approaching a Teammate’s PR Efficiently and Effectively
Review in a timely manner
- Once you are added as a reviewer on a PR, try to review it as soon as possible.
- The maximum time from a request and the first review being completed should never exceed one work day.
Gather context if it’s lacking
- If the PR is lacking context, or a description of its changes, reach out to the writer and get clarification before reviewing. This can significantly reduce back and forth caused by misunderstandings of the code or purpose.
Start at a high level
- Review from the high level down. Hold off on commenting on small changes like style until the larger issues, if they are present, are addressed. This way the author can focus on the most important revisions first. Sometimes the smaller issues will go away along with the larger changes.
- This also cuts down on the possibility of bike-shedding, which detracts from the more important and impactful aspects of the work.
Handling disagreements
- If you disagree with a code block, provide a code example of how you would suggest it be done instead. This is far more useful to the other developer than just leaving a the comment, “this should be different”.
Requesting changes
- Don’t block a PR unless there are fundamental issues with the work. Putting a roadblock in front of your coworker over an argument around naming doesn’t help anyone on the team. Give your teammate the opportunity to address the smaller changes and move on with their tasks without waiting for another round of feedback.
- Small issues should be addressed through a project style guide — but that’s a different topic altogether.
Empathy
- Most importantly, remember there’s another person on the end of your review. Provide feedback in a helpful and constructive way. Always think about how you would want to be given feedback, if it was your pull request that was being reviewed.
下载开源日报APP：https://opensourcedaily.org/2579/
加入我们：https://opensourcedaily.org/about/join/
关注我们：https://opensourcedaily.org/about/love/