Commit 8c118145 authored by Craig Norris's avatar Craig Norris

Merge branch 'tacker-dev-guidelines' into 'master'

Update awkward phrasing in dev guidelines

See merge request gitlab-org/gitlab!50187
parents 6baa1072 b68a6b4c
...@@ -72,8 +72,8 @@ Complementary reads: ...@@ -72,8 +72,8 @@ Complementary reads:
### Development guidelines review ### Development guidelines review
When you submit a change to GitLab's development guidelines, the people When you submit a change to GitLab's development guidelines, who
you ask for reviews from depend on the level of change, as described below. you ask for reviews depends on the level of change.
#### Wording, style, or link changes #### Wording, style, or link changes
...@@ -88,7 +88,7 @@ maintainer or Technical Writer. These can include: ...@@ -88,7 +88,7 @@ maintainer or Technical Writer. These can include:
#### Specific changes #### Specific changes
If the MR proposes changes limited to a particular stage, group, or team, If the MR proposes changes that are limited to a particular stage, group, or team,
request a review and approval from an experienced GitLab Team Member in that request a review and approval from an experienced GitLab Team Member in that
group. For example, if you're documenting a new internal API used exclusively by group. For example, if you're documenting a new internal API used exclusively by
a given group, request an engineering review from one of the group's members. a given group, request an engineering review from one of the group's members.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment