mirror of https://gitlab.com/gitlab-org/cli.git
docs: add development process
This commit is contained in:
parent
91ff651dd2
commit
4c7e995860
|
@ -0,0 +1,33 @@
|
|||
---
|
||||
stage: Create
|
||||
group: Code Review
|
||||
info: To determine the technical writer assigned to the Stage/Group associated with this page, see https://about.gitlab.com/handbook/product/ux/technical-writing/#assignments
|
||||
---
|
||||
|
||||
# Development process
|
||||
|
||||
This document describes how we make changes to the CLI. For contributing guidelines,
|
||||
read [`CONTRIBUTING.md`](../CONTRIBUTING.md).
|
||||
|
||||
## Who can make changes?
|
||||
|
||||
**Anyone** can make changes. If you are planning larger changes that affect the
|
||||
CLI architecture or dependencies,
|
||||
[create an issue first](https://gitlab.com/gitlab-org/cli/-/issues/new?issuable_template=Feature%20Request).
|
||||
|
||||
## Who reviews the changes?
|
||||
|
||||
The CLI project has [reviewer roulette](https://gitlab.com/gitlab-org/cli/-/merge_requests/1093).
|
||||
Each merge request receives a comment including a suggested reviewer and maintainer.
|
||||
Assign the reviewer first.
|
||||
|
||||
The rules for merging:
|
||||
|
||||
- **Merge requests created by maintainers**: One review from a reviewer, then the
|
||||
author maintainer may merge the merge request.
|
||||
- **Merge requests not created by maintainers**: One review from a reviewer, and one from a maintainer.
|
||||
|
||||
## Who releases the changes?
|
||||
|
||||
Only [project maintainers](https://gitlab.com/gitlab-org/cli/-/project_members?sort=access_level_desc)
|
||||
can tag a release. Follow the [release process](release_process.md).
|
Loading…
Reference in New Issue