Changes to this repository are first reviewed using the merge request workflow then merged by project maintainers.
Architectural decisions (such as those that would appear on the architecture or decisions pages) require the review of the project’s senior technical leadership. Senior technical leadership are individuals identified by the Engineering Manager of the team responsible for the project, as well as that team’s Staff+ leadership as mentioned in the architecture handbook and any current working group formed around a goal specific to the project.
Project maintainers can be found on the GitLab projects page, or located using the review workload dashboard.
Maintainers are responsible for merging changes within their domain, and having an understanding of the whole project and how changes may impact areas outside their expertise.
Reviewers can assign to any maintainer and the maintainer will engage the appropriate domain expert if it does not fall within their own.
In order to continue to expand their expertise maintainers are empowered to merge changes outside their domain but that they are highly confident in unless:
- The change cannot be reverted later
- The change has an established process that needs to be followed (JiHu review, security, legal/license changes)
- The change clearly requires an architectural decision
When urgent changes are required, maintainers should have a bias-for action, and can make decisions as long as the decisions are later reversible and compliant with known project process requirements.
|@WarheadsSE||Staff Engineer, Distribution Deploy|
|@twk3||Engineering Manager, Distribution Build|
|@ayufan||Distinguished Engineer, Enablement|