Skip to content

Commit 124663c

Browse files
committed
fixup: line length
Signed-off-by: Artem Gindinson <gindinson@roofline.ai>
1 parent 583203e commit 124663c

1 file changed

Lines changed: 14 additions & 13 deletions

File tree

docs/website/docs/developers/general/contributing.md

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -158,21 +158,22 @@ project components.
158158

159159
The
160160
[`.github/CODEOWNERS` file](https://github.com/iree-org/iree/blob/main/.github/CODEOWNERS)
161-
designates reviewers for specific components of the project. Maintainers of the project
162-
are assigned as reviewers by default, so that they serve as the first point of contact
163-
and help find a reviewer in case they can't perform the review themselves.
161+
designates reviewers for specific components of the project. Maintainers of the
162+
project are assigned as reviewers by default, so that they serve as the first
163+
point of contact and help find a reviewer in case they can't perform the review
164+
themselves.
164165
165166
A strong, recent history of quality commits to a certain part of the project is
166-
generally enough to nominate oneself as a CODEOWNER - subject to maintainer approval,
167-
such participation in the project would be very welcome.
168-
169-
An approval on the PR generally doesn't have to come from a designated code owner.
170-
A single approval by a project member with write access is technically
171-
sufficient to merge the change, however authors of larger PRs that affect multiple components
172-
should allow more time for several code owners and other project members to review.
173-
In such cases, preceding the change with a RFC and aligning with code owners/maintainers
174-
on the design is also a recommended practice - this allows to minimize time spent in the
175-
final PR review stage.
167+
generally enough to nominate oneself as a CODEOWNER - subject to maintainer
168+
approval, such participation in the project would be very welcome.
169+
170+
An approval on the PR generally doesn't have to come from a designated code
171+
owner. A single approval by a project member with write access is technically
172+
sufficient to merge the change, however authors of larger PRs that affect
173+
multiple components should allow more time for several code owners and other
174+
project members to review. In such cases, preceding the change with a RFC and
175+
aligning with code owners/maintainers on the design is also a recommended
176+
practice - this allows to minimize time spent in the final PR review stage.
176177

177178
## :octicons-code-16: Coding policies
178179

0 commit comments

Comments
 (0)