Gitlab and plasma dev

[Kai Uwe Broulik]

Hi,

 > I think this could be achieved with a label, that reviewers can set 
and authors can clear when they're ready.
 > Somehow it's possible to create a label that covers all of Plasma as 
the VDG have one already.

Good idea! I think having a "Needs changes" label would be very useful, 
even on a global scale, not just for Plasma. Then you could also filter 
those ones out from your search queries.

However, given GitLab has this concept of resolved threads where it 
wouldn't let you merge until everything's remedied - maybe there already 
exists infrastructure we could leverage for this purpose? (probably only 
in EE, scnr)

What I would like to see is a "VDG approved" or "Design approved" label 
to explicitly express for designers "we like the looks of it" or "the 
interaction pattern is sound" but that someone still needs to review the 
actual code.

Nate's been doing this on Phabricator by doing this "Accepted by VDG" 
which I recall was an awkward dance of clicking accept and then 
unchecking oneself. Making this more explicit and user-friendly would be 
appreciated.

Cheers
Kai Uwe