Design reviews are mandatory for every merge request in these projects:
All designers are maintainers of the
gitlab-design project. If you are interested in becoming a Maintainer of UI (
gitlab-ui projects, please follow the Engineering Review Workflow.
You can find all design reviewers and maintainers by looking on the team page and filtering by
UX Department or on the list of GitLab Engineering Projects, both of which are fed by
These guidelines describe who would need to review, approve, and merge your (or a community member's) merge request.
All GitLab designers can (and are encouraged to) perform design and code reviews on merge requests that impact product design. This includes contributions from GitLab Team Members and the wider GitLab community. If you want to review merge requests, you can wait until someone assigns you one, but you are also more than welcome to browse the list of open merge requests and leave any feedback or questions you may have.
Note that while all designers can review all merge requests, the ability to accept merge requests is restricted to maintainers.
Maintainers are GitLab designers who:
Every project has at least one maintainer, but most have multiple, and some projects (like gitlab-ui and design.gitlab.com) have separate maintainers for design and frontend.
Read more about what makes great maintainers in the Engineering Review Workflow.
Design maintainers are divided into types of maintainership. This helps maintainers specialize, but more importantly, it helps speed up the process of becoming a maintainer, and with knowing who to ask for final acceptance of merge requests. New types can be created or existing ones refactored to accommodate the natural evolution of projects.
We follow the same maintainer guidelines as our Engineering counterparts. Get familiar with those guidelines and how to become a maintainer in the Engineering Review Workflow.
Three key aspects of that process:
We're not able to support more trainees at the moment. Even so, we still encourage people to nominate themselves if they're interested! We'll accomodate them once we have a Support Maintainer available for them.
We follow the same trainee maintainer program (traineeship) as our Engineering counterparts. Anyone may nominate themselves as a trainee by opening a tracking issue using the Trainee design maintainer template.
For the traineeship, a design maintainer is assigned as Support Maintainer to each trainee. The Support Maintainer will direct merge requests for review, give feedback on proposals, and discuss process or progress during 1:1 sessions. Trainees can always reach out to their managers or other maintainers for feedback and guidance separate from the dedicated Support Maintainer. We've also created a Pajamas UX maintainer review checklist to help guide trainee maintainers as they learn to review Pajamas MRs.
We try to keep a maximum of one trainee per maintainer, so that maintainers are not overloaded with their additional role as Support Maintainers and can provide adequate guidance. See our current trainee maintainers.
The traineeship is a long commitment, usually several months, and takes away time from other responsibilities. If you're interested in enrolling in this program, please talk with your manager and team before nominating yourself, as the traineeship is likely to impact your capacity.
There are two aspects that play a big part in the duration of the traineeship: the number of hours that are dedicated to it and the number of available merge requests for the trainee. When these two aspects oppose each other, the traineeship can take longer than expected:
To help track progress, we encourage trainees to make the traineeship one of their personal OKRs.
See the Pajamas maintainer ratio dashboard.
|Pajamas Design System||Jarek Ostrowski||Taurie Davis|
|Pajamas Design System||Amelia Bauerly||Rayana Verissimo|
|Pajamas Design System (Figma)||Nadia Sotnikova||Jeremy Elder|
|Pajamas Design System (Figma)||Michael Le||Jeremy Elder|
|Pajamas Design System (Figma)||Libor Vanc||Jeremy Elder|
|GitLab SVGs||Taurie Davis||Jeremy Elder|