summaryrefslogtreecommitdiff
path: root/contributors
diff options
context:
space:
mode:
authorJorge O. Castro <jorge.castro@gmail.com>2018-01-23 09:13:57 -0500
committerJorge O. Castro <jorge.castro@gmail.com>2018-01-23 09:13:57 -0500
commit6d51ca785dadf530ff833d2d3c015a226e09f29e (patch)
tree37672bd6f75e213b8af7aacb2f546079115844ef /contributors
parent774750b5d22b16203a0ea18d352821cb0aa7d43a (diff)
Remove duplicate content and fix links
Diffstat (limited to 'contributors')
-rw-r--r--contributors/guide/community-expectations.md30
1 files changed, 4 insertions, 26 deletions
diff --git a/contributors/guide/community-expectations.md b/contributors/guide/community-expectations.md
index 39d2da73..eaa1787d 100644
--- a/contributors/guide/community-expectations.md
+++ b/contributors/guide/community-expectations.md
@@ -22,36 +22,14 @@ As a community we believe in the value of code review for all contributions.
Code review increases both the quality and readability of our codebase, which
in turn produces high quality software.
-However, the code review process can also introduce latency for contributors
-and additional work for reviewers that can frustrate both parties.
+See the [pull request documentation](/devel/pull-requests.md) for more information
+on code review.
Consequently, as a community we expect that all active participants in the
community will also be active reviewers. The
-[community membership](../community-membership.md) outlines the responsibilities
+[community membership](/community-membership.md) outlines the responsibilities
of the different contributor roles.
-All changes must be code reviewed. For non-maintainers this is obvious, since
-you can't commit anyway. But even for maintainers, we want all changes to get at
-least one review, preferably (for non-trivial changes obligatorily) from someone
-who knows the areas the change touches. For non-trivial changes we may want two
-reviewers. The primary reviewer will make this decision and nominate a second
-reviewer, if needed. Except for trivial changes, PRs should not be committed
-until relevant parties (e.g. owners of the subsystem affected by the PR) have
-had a reasonable chance to look at PR in their local business hours.
-
-Most PRs will find reviewers organically. If a maintainer intends to be the
-primary reviewer of a PR they should set themselves as the assignee on GitHub
-and say so in a reply to the PR. Only the primary reviewer of a change should
-actually do the merge, except in rare cases (e.g. they are unavailable in a
-reasonable timeframe).
-
-If a PR has gone 2 work days without an owner emerging, please poke the PR
-thread and ask for a reviewer to be assigned.
-
-Except for rare cases, such as trivial changes (e.g. typos, comments) or
-emergencies (e.g. broken builds), maintainers should not merge their own
-changes.
-
Expect reviewers to request that you avoid [common go style
mistakes](https://github.com/golang/go/wiki/CodeReviewComments) in your PRs.
@@ -61,7 +39,7 @@ Because reviewers are often the first points of contact between new members of
the community and can significantly impact the first impression of the
Kubernetes community, reviewers are especially important in shaping the
Kubernetes community. Reviewers are highly encouraged to review the
-[code of conduct](../../governance.md#code-of-conduct) and are strongly
+[code of conduct](/governance.md#code-of-conduct) and are strongly
encouraged to go above and beyond the code of conduct to promote a collaborative,
respectful Kubernetes community.