From 3286007fb97b0d69c2de52fa24299cbfd0d0dce3 Mon Sep 17 00:00:00 2001 From: Ryan Hill Date: Mon, 24 Aug 2026 16:29:35 -0500 Subject: [PATCH 1/2] Add CODEOWNERS designating @ryanhill1 --- .github/CODEOWNERS | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 .github/CODEOWNERS diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS new file mode 100644 index 0000000..56b82fd --- /dev/null +++ b/.github/CODEOWNERS @@ -0,0 +1,6 @@ +# This file is being listed as code owner means the user +# will be empowered to approve and merge pull request for +# that directory of code. +# +# qBraid algorithms maintainers +* @ryanhill1 From 3d01db40a77e7346aa23b3aa69bf1cb0cdf89a58 Mon Sep 17 00:00:00 2001 From: Ryan Hill Date: Mon, 24 Aug 2026 17:28:44 -0500 Subject: [PATCH 2/2] docs: correct what CODEOWNERS actually does The header claimed that being listed here empowers a user to approve and merge pull requests for that directory. None of that is right. CODEOWNERS routes review requests; it grants no permissions. Whether an owner's approval gates a merge depends on a branch protection rule or ruleset enabling "require review from Code Owners", and the ability to merge at all comes from the user's repository role. The pattern is also `*`, which matches every path in the repository rather than a directory. Comment only -- the `* @ryanhill1` rule is unchanged. --- .github/CODEOWNERS | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 56b82fd..da1426e 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -1,6 +1,10 @@ -# This file is being listed as code owner means the user -# will be empowered to approve and merge pull request for -# that directory of code. +# Owners listed here are requested for review automatically when a pull request +# touches a path they own. Ownership confers no permissions by itself: whether an +# owner's approval is required before merging depends on a branch protection rule +# or ruleset enabling "require review from Code Owners", and the ability to merge +# comes from a user's repository role. +# +# The pattern below matches every path in the repository. # # qBraid algorithms maintainers * @ryanhill1