Skip to content

Help reduce async MR review cycles by clarifying non-blocking

Doug Stull requested to merge reduce-async-cycles-in-review into master

What does this MR do and why?

Help reduce async MR review cycles by clarifying non-blocking

  • We find sometimes that first level reviewers and maintainers are unclear how to handle non-blocking suggestions. A lot of times a maintainer will make a non-blocking comment and pass it back for another async cycle to the MR author instead of setting it to merge. When there are only non-blocking suggestions, this can be inefficient. The aim of this change is to clear that up and further enable first level reviewers and maintainers to reduce those async cycles and empower creation of follow-up issues.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports