Also known as Merge Requests.
Pull requests tell other team members that you changed something in the code and pushed the change to a branch in a git repository.
What Is a Pull Request
A pull request is a practice of getting feedback from other programmers and deciding to "merge" or "do not merge" the code before it is merged into the main codebase.
If "do not merge decision" is made, author of the code can address the comments of others and re-submit the code for another round of review. This continues until all issues with code are resolved and the code can be merged. Or until the idea behind the code is no longer in alignment with the project goals and the pull request is rejected. For more information, see the Code Review practice.
Pull request can also trigger CI/CD to make sure all automated tests pass before the code can be merged. For more infomration, see Continuous Integration and Continuous Delivery.
Source: Tuleap: Pull Request
Why You Might Want the Pull Request?
- Using pull request helps you catch the bad code before it is merged into the main codebase.
- Author of the code can also gain valuable feedback on his work that can be immediately applied when fixing the pull request.
- The knowledge of the codebase is spread more evenly in the team when pull requests practice is in place.
- The Team Leader or the senior colleague can use pull requests to continually evaluate performances of other team members.
Problems the Pull Request Helps to Solve
- Poor code quality
- Demotivated Team
- Knowledge hoarding
- Toxic Team Culture
- "Not my problem" mentality
- Meaningless work
- Toxic team culture
How to Implement the Pull Requests
- Explain the pull requests practice to the team, have a discussion about benefits vs. drawbacks the team can be afraid of.
- Adjust the workload for the team so they have enough time for doing the pull requests.
- Make rules on who will be reviewing what. The rules will be depending on your goals.
- If knowledge sharing is a priority, assign the pull request to someone who does not know this part of the codebase.
- If a code quality is a priority, let the more senior team members do the review of the more junior members' code. Senior members can do the Code Review with each other.
- If culture is especially toxic, a simple round-robin style of pull request assignment can be seen as “fair”.
- Rules can be usually enforced by Version Control System (CVS).
- Demonstrate to the team the pull request flow in your CVS.
- Start practicing pull requests within the team.
- Continuously give feedback to the team, fine-tune the rules so it fits the team.
- Encourage short pull requests, preferably with not more than 250 lines changed. Large code changes make finding bugs much more demanding and hinder the code review process. A way to minimize large pull requests is to break down broad features into smaller independent tasks that your dev team can solve individually. Small pull requests allow multiple developers to work on issues simultaneously and thus speed up product development.
- Practice having relevant titles and descriptions for your pull requests. The pull request description should always explain why the person created the pull request, what issues it solves, and how. The best pull request should also mention relevant people. Additionally, adding a screenshot or GIF to demonstrate the visual changes can go a long way in simplifying code review.
- Add a pull request template to the root directory of your repositories. Alternatively, you can also add it to docs/ ,PULL_REQUEST_TEMPLATE/ or .github/. Pull request templates are markdown or simple text files that get added to the pull request description automatically. Name the file pull_request_template.md and include the formatting that you want to use in your pull requests. Using templates makes code review smoother by standardizing pull requests and also helps inexperienced developers write good pull requests.
Common Pitfalls of Pull Requests
- Make sure that the pull requests are done regularly in your team and that nobody waits way too long for their code to be reviewed. If the code is left unchecked, it can lead to “long feedback loops” and it can effect team motivation.
- Always have at least two possible approvers for each pull request. Having only one person that can merge pull requests could lead to power hoarding.
Resources for Pull Requests
Want to write for DXKB?
Feel free to contribute. People from DXKB community will be more than happy.
Continuous Integration is a software development practice that makes developers integrate code changes into a shared repository routinely and frequently. Usually, each person integrates at least daily and that ensures them that their code changes do not break anything.Read more
A Bus Factor measures the minimum number of team members who have to be hit by a bus to put the project in jeopardy. The goal is to increase your Bus Factor as much as possible.Read more
Release management is the process of managing, planning, designing, scheduling, testing, controlling and deploying of a software build through different stages and environments; in preparation for software releases.Read more
Code Review is an important practice for checking each other's code. The reviewers are other developers from the team. The goal is to uncover potential mistakes that could slip through testing.Read more
Git Flow is a specific branching system for Git. It helps the team to better control and add different project versions.Read more