From e9b25a1e98b10f5f578a7765573607ccba0ac565 Mon Sep 17 00:00:00 2001 From: Sonu Kapoor Date: Thu, 23 Jul 2020 10:50:08 -0400 Subject: [PATCH] docs: remove duplicate `https://` (#38199) This doc contained a duplicate `http://` before the domain name leading to an invalid link. This commit fixes this issue. PR Close #38199 --- docs/TRIAGE_AND_LABELS.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/TRIAGE_AND_LABELS.md b/docs/TRIAGE_AND_LABELS.md index 8251812ac3..aa23e674c4 100644 --- a/docs/TRIAGE_AND_LABELS.md +++ b/docs/TRIAGE_AND_LABELS.md @@ -175,7 +175,7 @@ If a PR is missing the `PR target: *` label, or if the label is set to "TBD" whe Before a PR can be merged it must be approved by the appropriate reviewer(s). -To ensure that the right people review each change, we set review requests using [PullApprove](https://https://docs.pullapprove.com/) (via `.pullapprove`) and require that each PR has at least one approval from an appropriate code owner. +To ensure that the right people review each change, we set review requests using [PullApprove](https://docs.pullapprove.com/) (via `.pullapprove`) and require that each PR has at least one approval from an appropriate code owner. If the PR author is a code owner themselves, the approval can come from _any_ repo collaborator (person with write access). In any case, the reviewer should actually look through the code and provide feedback if necessary.