Skip to content

Adopt new UX review roulette

Lin Jen-Shin requested to merge update-danger-for-ux into master

What does this MR do and why?

Adopt new UX review roulette from gitlab-org/ruby/gems/gitlab-dangerfiles!212 (merged)

Gem was released and updated at !124492 (merged)

Screenshots or screen recordings

When it's Community contribution without a group

Pedro was selected: https://gitlab.com/pedroms

Screenshot_2023-06-15_at_9.50.38_PM

When it's Community contribution + groupsource code

The designer for source code was selected: https://gitlab.com/mle

Screenshot_2023-06-15_at_10.29.53_PM

When it's just Engineering Productivity which doesn't have a designer

No UX reviewer was selected:

Screenshot_2023-06-15_at_10.36.10_PM

When it's groupsource code

A designer who is not for source code was selected: https://gitlab.com/danmh

Screenshot_2023-06-15_at_10.40.30_PM

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Lin Jen-Shin

Merge request reports