Flags deep dive: PR has process defects

Modified on Mon, 3 Feb at 12:52 PM

When to use this Flag?

Leverage this flag to identify all PRs that have process defects e.g. reviews, approvals, merge conflicts, deviations from number of approvals before merge etc.


Impact:

Process

Classification:

Code

Tools:

Atlassian Jira, Github, Gitlab, Bitbucket

Intent:

The intent of this flag is to reduce the time spent by tech leads and engineering managers in reviewing PRs to detect process defects that can lead to development bottlenecks, cause delays, and ultimately impact product quality. At times PRs that go undetected create documentation hassles and it takes a lot of time to extract their logs. Proactively identifying PRs with process defects can also avoid such situations. 


Configurations:


  1. Configure the following for an open/merged PR:
    1. Configure the thresholds for Open PRs and Merge PRs (in terms of the creation of the PR).
    2. Enable the "PR Approval check" condition to flag a open PR where the PR was approved without any review comments provided by the reviewer.
    3. Enable the "PR review comment check" condition (with the configured threshold) to flag a open PR, if the author has not addressed comment as per the configured threshold.
    4. Enable the "PR update check" condition (with the configured threshold) to flag an open PRs if the PR was not modified beyond the configured threshold (represents a stale PR).
    5. Enable the "PR merge conflict check" condition to flag a open PR, if the PR has merge conflicts.
    6. Enable the "PR merge approver count check" condition to flag a merged PR, if the PR had less number of approvers than the configured threshold.
  2. Enable auto-resolution settings and select "Move issue(s) to sprint" as the option, this will ensure that the platform automatically moves all issues (breaking the threshold) the next planned sprint.
  3. Enable this as a compliance flag if you think this can have an impact on your internal or external regulations; this will ensure the visibility of all the artefacts violating this flag in the compliance status report.


.

Was this article helpful?

That’s Great!

Thank you for your feedback

Sorry! We couldn't be helpful

Thank you for your feedback

Let us know how can we improve this article!

Select at least one of the reasons

Feedback sent

We appreciate your effort and will try to fix the article