[BuildStream] Proposal: label name change: review -> verify

[Agustín Benito Bethencourt <agustin benito codethink co uk>]

Dear BuildStream fans,

there seem to be a confusion about what the label Review which is described in 
the policy[1] as a state label:

"Review: items that are under review once the developer or contributor has 
finished it."

I propose the change the label name from "Review" to "Verify" avoid any 
misunderstanding.

The Status scale would be:

* Backlog: default state on Gitlab so no label needed.
* Todo: processed elements that should be done in the future.
* Doing: WIP
* Verify: items that has been finished but somebody else should go over them to 
check them before closing them.
* Closed: items that has been processed and canceled, finished or no longer 
apply. Default state on Gitlab so no label needed.

In summary, a ticket should be in Verify state when its acceptance criteria 
has been met and somebody else needs to check that it is the case.

Please do not mistake it with... this ticket includes a MR that needs to be 
reviewed. Calling the attention of the reviewers in order to have your new 
code reviewed should be made through the MR, not the related issue.

[1] https://gitlab.com/BuildStream/nosoftware/alignment/blob/master/
BuildStream_policies.md 

Best Regards
-- 
Agustín Benito Bethencourt
Principal Consultant
Codethink Ltd
We respect your privacy.   See https://www.codethink.co.uk/privacy.html