[virglrenderer-devel] reviewing merge requests

[Gert Wollny]

Hi all, 

since Dave promoted me to maintainer I'm now try to take a look to see
what MRs can be merged, but sometimes it is just inconvenient to do
this every day, so I'd like to I ask the following favours to make
things more smooth: 

* When you submit MRs, please enable the CI, I prefer to merge using
  "Merge when pipeline passes"

* After doing a review please tag the MR with one of 
   - is-reviewed,
   - waiting for changes, or  
   - needs more reviews
  with that I can see the state of a MR without digging through the 
  comments or through all the emails.   

* when you've done a review and you add the RB-tag 
  - could you pleaseformat the tag as code for easy copy-and-paste? 

* When you got a review on your own MR and have time, then it is nice
  if you could add the R-B tags to the commit messages and add the tag 
  - rb-tags added 

* When you have Developer state, 
  - when you git a review, don't wait for me to merge your own on MRs, 
    Me or Dave will probably eventually do it, but  but it's faster if 
    you do it yourself,
  - if you reviewed a MR by someone who can't merge, feel free to merge
    it yourself, 
  - using the "Merge when  pipeline passes" button is preferred 
   
Many thanks, 
Gert