I think exploring these additions to PR reviews would be pretty helpful. It’s worth spending the extra time to get a PR right before merging.
Anthony > On Oct 28, 2020, at 8:40 AM, Robert Houghton <rhough...@vmware.com> wrote: > > There are some pieces of Apache infrastructure we can control without needing > tickets: Specifically > required_pull_request_reviews: > dismiss_stale_reviews: true > require_code_owner_reviews: true > > I think these specific controls could go a long way towards helping keep our > PR quality high, while reducing cognitive load for the reviewers. > > Full documentation > here<https://cwiki.apache.org/confluence/display/INFRA/git+-+.asf.yaml+features#git.asf.yamlfeatures-BranchProtection> >
