Skip to content

Change Review priority label meaning

Steve Xuereb requested to merge change-review-priority into master

What does this MR do?

Change Review priority label meaning

Why was this MR needed?

ReviewP3 is now the default label for all incoming Merge Requests, whilst ReviewP2 are for high impacting Merge Requests but not as much as when compared to ReviewP1

This idea cam when we started discussing !1164 we said this is the highest ReviewP2 but not quite ReviewP1 because we want don't want to breach out SLO for ReviewP1 where we have more than 1 ongoing Merge Request.

This gives us finer control of what is "next" after we finish the ReviewP1 or if they are blocked we can start working our way through ReviewP2

Merge request reports