How Developers Modify Pull Requests in Code Review

Jiang, Jing; Lv, Jiangfeng; Zheng, Jiateng; Zhang, Li

doi:10.1109/tr.2021.3093159

Cited by 2 publications

(1 citation statement)

References 45 publications

Supporting

Mentioning

Contrasting

Order By: Relevance

“…The PR then undergoes a review process, during which the contributor revises the changes based on feedback from the project maintainers. This cycle repeats until the PR satisfies the maintainers' requirements for getting merged [3,4].…”

Section: Introductionmentioning

confidence: 99%

On Wasted Contributions: Understanding the Dynamics of Contributor-Abandoned Pull Requests–A Mixed-Methods Study of 10 Large Open-Source Projects

Khatoonabadi

Costa

Shihab

2023

ACM Trans. Softw. Eng. Methodol.

View full text Add to dashboard Cite

Pull-based development has enabled numerous volunteers to contribute to open-source projects with fewer barriers. Nevertheless, a considerable amount of pull requests (PRs) with valid contributions are abandoned by their contributors , wasting the effort and time put in by both the contributors and maintainers. To better understand the underlying dynamics of contributor-abandoned PRs, we conduct a mixed-methods study using both quantitative and qualitative methods. We curate a dataset consisting of 265,325 PRs including 4,450 abandoned ones from ten popular and mature GitHub projects and measure 16 features characterizing PRs, contributors, review processes, and projects. Using statistical and machine learning techniques, we find that complex PRs, novice contributors, and lengthy reviews have a higher probability of abandonment and the rate of PR abandonment fluctuates alongside the projects’ maturity or workload. To identify why contributors abandon their PRs, we also manually examine a random sample of 354 abandoned PRs. We observe that the most frequent abandonment reasons are related to the obstacles faced by contributors, followed by the hurdles imposed by maintainers during the review process. Finally, we survey the top core maintainers of the studied projects to understand their perspectives on dealing with PR abandonment and on our findings.

show abstract

Section: Introductionmentioning

confidence: 99%