From 7b0999303a2f4ccf4267c98f30489666909b1c62 Mon Sep 17 00:00:00 2001 From: Matthias Fasching <5011972+fasmat@users.noreply.github.com> Date: Wed, 12 Jun 2024 20:33:16 +0000 Subject: [PATCH] Update CODEOWNERS (#6037) ## Motivation Update CODEOWNERS to include @acud --- .github/CODEOWNERS | 8 ++++++-- .github/workflows/ci.yml | 4 +++- 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index cc4048d817..ea63a46113 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -1,3 +1,7 @@ -# CODEOWNERS: https://help.github.com/articles/about-codeowners/ +# Code owners +# For details see: https://help.github.com/articles/about-codeowners/ -* @dshulyak @fasmat @poszu @ivan4th +# Default owners for the repository. +# Unless further specified all of those will be requested for review whenever someone opens a PR. +# These are also the only users who are allowed to `bors try` and `bors merge`. +* @dshulyak @fasmat @poszu @ivan4th @acud diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 5dac8aadde..b841cf1240 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -46,10 +46,12 @@ jobs: with: # this pattern matches using picomatch syntax (used by this third party Action), which is slightly # different than GitHub syntax: it matches any file in any path ending in '.md'. this checks if - # any non-markdown files were changed. + # any non-markdown files were changed. Additionally changing the CODEOWNERS file should not trigger + # a full CI run. filters: | nondoc: - '!**/*.md' + - '!.github/CODEOWNERS' ## stage 1: run unit tests and app tests as a prerequisite ## these run on all pushes to all pull requests, all branches