Skip to content

Commit

Permalink
Update CODEOWNERS (#7267)
Browse files Browse the repository at this point in the history
Removing @joeflack4 as codeowner so he does not get auto-assigned to every PR (especially draft ones)

We changed branch protection settings so non-owners can also approve PRs

Co-authored-by: Trish Whetzel <[email protected]>
  • Loading branch information
matentzn and Trish Whetzel authored Feb 14, 2024
1 parent 7250b92 commit c0b01e5
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@

# Removed @cmungall to stop him being added to all PRs
* @sabrinatoro @twhetzel
*.sparql @joeflack4 @twhetzel

This comment has been minimized.

Copy link
@joeflack4

joeflack4 Feb 14, 2024

Collaborator

This CODEOWNERS file contains errors …

Unknown owner on line 11: make sure a user with the email address [email protected] exists and has write access to the repository
docs/*  [email protected]
Invalid owner on line 12:
docs/*  actions-user

Thanks for this! I'd be happy to review a SPARQL change if it's relevant and I'm tagged manually.

@matentzn @twhetzel I just saw this error; I imagine you guys saw it as well, but flagging just in case you did not.

This comment has been minimized.

Copy link
@twhetzel

twhetzel Feb 14, 2024

Collaborator

Thanks, I did not see that. Looks like this error has existed for at least 3 years so would be good to sort out, but lowest on priority.

This comment has been minimized.

Copy link
@joeflack4

joeflack4 Feb 14, 2024

Collaborator

that's a long time lol
I would create an issue but now that I think of it, Nico has told me before that he is aware of some issue with the docs GitHub action so perhaps this is already somewhere in some issue

*.py @joeflack4 @twhetzel
*.md @sabrinatoro @twhetzel
*.sparql @twhetzel
*.py @twhetzel
*.md @sabrinatoro @twhetzel @nicolevasilevsky
Makefile @matentzn @twhetzel
mondo.Makefile @matentzn @twhetzel

Expand Down

0 comments on commit c0b01e5

Please sign in to comment.