You are here:

Self Merge Rates

Question: How many of the contributions in the project repository are being merged by the original contributor?

Description

This metric measures the number of change requests merged by the author and merged without going through a change request review process.

Objectives

Healthy and collaborative projects will have a culture of doing reviews of all contributions regardless of who – from first-time contributors to project creators–made the contribution in the repository. Including reviews in any changes to the code reinforces the merge process for newcomers and ensures that another person has seen and agreed with the change to be made. Ideally, you should not see any self merges or merges without review in a repository unless there is a valid reason that is agreed to in the community (e.g., a simple typo fix).

  • Community managers would be interested in seeing if a community is making progress in their review health. (e.g., did the self merge rate go down this year vs. last year?)
  • Community managers would be interested in seeing if the number of community members involved in change requests/reviews is changing over time.
  • OPSO managers want to understand what open source projects to incorporate into their solutions by evaluating project’s review health.
  • Anyone (e.g., potential users or community members) interested in evaluating the health of the community would want to see the self merge and merge without review rate trends over time and see if there's a healthy review of contributions in the repository.

Implementation

Filters

  • Merge without review
  • Lines of code changed
  • File type changed (e.g., code vs. documentation)
  • Volume/size of reviews
  • Time

Visualizations

Visualization #1: Image showing the screenshot of reviewers in GitHub

Reviewers in GitHub

Visualization #2: Image showing the screenshot of reviewers in GitLab

Reviewers in GitLab\

Tools Providing the Metric

  • GitHub interface
  • GitLab interface

Data Collection Strategies

  • Existence of automated reviews (e.g., non-bot and non-rule based reviews): there should be reviews with text-based comments, emoji’s, or assigned reviewer approval.

References

Known Contributors

  • Ray Paik
  • Matt Germonprez
  • Kevin Lumbard
  • Elizabeth Barron
  • Dawn Foster

To edit this metric please submit a Change Request here: https://github.com/chaoss/wg-common/blob/main/focus-areas/contributions/self-merge-rates.md

To reference this metric in software or publications please use this stable URL: https://chaoss.community/?p=5306

The usage and dissemination of health metrics may lead to privacy violations. Organizations may be exposed to risks. These risks may flow from compliance with the GDPR in the EU, with state law in the US, or with other laws. There may also be contractual risks flowing from terms of service for data providers such as GitHub and GitLab. The usage of metrics must be examined for risk and potential data ethics problems. Please see CHAOSS Data Ethics document for additional guidance.

Tags:
Was this article helpful?
Dislike 0