Pull Requests (PRs) are an essential part of software development as they facilitate the review and merging of code changes. However, if not managed effectively, PRs can contribute to work wastage in development teams. In this article, we will explore how engineering managers can use PRs charts to reduce work wastage and optimize their development processes.
The Problem of Work Wastage in Development Teams
Work wastage refers to the unnecessary effort and time spent on tasks that do not contribute to the final product. In software development, work wastage can lead to increased costs. PRs can also contribute to work wastage when not managed effectively. For instance, when PRs are closed without merging, the effort put into creating and reviewing them is wasted. Therefore, it is essential to minimize the number of PRs that are not merged to reduce work wastage in development teams.
Reasons for not merging PRs
Here are some reasons why a pull request (PR) may be closed without merging:
- Proof of concept changes: POC changes are by definition intended for collaborating with team members and there is no intention of merging them.
- Outdated PR: If the changes proposed in the PR become outdated due to other changes in the codebase, the PR may be closed without merging.
- Scope changes: If the scope of the task or feature changes significantly, the PR may be closed without merging and a new PR may be submitted to address the updated scope.
- Abandoned PR: If the PR author abandons the changes or does not respond to feedback or review comments for an extended period, the PR may be closed without merging.
- Duplicate PR: If there is already an open PR for the same change or feature, the duplicate PR may be closed without merging.
- Rejected PR: If the code changes proposed in the PR do not meet the quality standards, technical requirements, or development guidelines, it may be rejected and closed without merging.
PRs Charts
PRs charts are a powerful tool for engineering managers to track and analyze the performance of their development teams. These charts provide various data points and metrics that can help managers identify trends, patterns, and areas of improvement in their development processes.
One such chart is the PRs by type chart, which gives a breakdown of closed PRs by type. This chart provides insight into the percentage of PRs that were merged, closed but not merged, and closed earlier but reopened. By minimizing the PRs that are not merged, engineering managers can reduce work wastage of their development teams.
Using PRs Charts to Reduce Work Wastage
It is crucial to minimize the number of PRs that are not merged to reduce work wastage in development teams. Except for proof of concept prs that are created explicitly with the intent of not merging other prs were intended for merging but did not get merged leading to wastage of work. We should check pr by type charts and if unmerged prs are greater than certain percentage than we should have a closer look at the unmerged prs to identify their root cause. It may be indicative of problem in our process for example if we find the prs are not getting merged because of change in scope than we need to make sure that we improve our planning process or if prs are not getting merged due to change in dependencies than we may need to collaborate more closely with other teams. By imrpoving the process we can reduce the likelihood of unmerged PRs.
Coclusion
In conclusion, PRs are an essential part of software development, but if not managed effectively, they can contribute to work wastage in development teams. Using PRs charts to track and analyze the performance of development teams can help identify areas of improvement and reduce work wastage. It is essential for engineering managers to continually monitor their PRs charts to optimize their development processes.