From bd8b3a9046fd3ab0ac17e441824777ecafcaf259 Mon Sep 17 00:00:00 2001 From: "J. Ryan Stinnett" Date: Wed, 18 Mar 2020 10:21:28 +0000 Subject: [PATCH] Move multi-repo linking to workflow section --- docs/review.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/docs/review.md b/docs/review.md index cdf42b8bf..6c5472d2b 100644 --- a/docs/review.md +++ b/docs/review.md @@ -37,12 +37,12 @@ When reviewing code, here are some things we look for and also things we avoid: * Take the time to thank and point out good code changes * Using softer language like "please" and "what do you think?" goes a long way towards making others feel like colleagues working towards a common goal -* Authors should link to other layers of their PR in their PR before requesting - review. Reviewers might be coming from different places and could miss other - required PRs. ### Workflow +* Authors should link to other layers of their PR in their PR before requesting + review. Reviewers might be coming from different places and could miss other + required PRs. * Avoid force pushing to a PR after first round of review * Use merge commits when landing * PR author merges after review (assuming they have write access)