Skip to content

Commit 5a8fa4b

Browse files
committed
org-repo: update to reflect the more relaxed merge rules
See #218 for the change.
1 parent fa2bd9f commit 5a8fa4b

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

doc/org-repo.md

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,16 @@
11
# Organisation repo
22

33
This repository itself is the entry point for documentation on official resources.
4+
It is important that the documentation for the NixOS org governance is accurate.
45

5-
Everybody in the [CODEOWNERS](../.github/CODEOWNERS) file has write permission to this repository.
6-
This allows people to get automatic review requests and merge PRs for the files that concern them.
7-
PRs can only be merged if a codeowner for the respective files approves it, and all files need to have a codeowner entry.
6+
The [`CODEOWNERS`](../.github/CODEOWNERS) file indicates the persons or groups who have either ownership or a stake in a document.
7+
In addition to being a document in its own right, the named parties are added as an automatic review requests, and can merge PRs for the files that concern them.
8+
All files in this repository need to have an entry in the `CODDEOWNERS` file.
9+
This entry serves as documentation for who the interested and relevant parties are to that section of governance.
810

9-
Furthermore, the code owners for the CODEOWNERS file should have permission to give more people write access to this repository.
10-
These people get requested for reviews when new people add themselves to CODEOWNERS, allowing them to give write access when merged.
11+
The [GitHub org owners](./github-org-owners.md) are themselves the codeowner for the `CODEOWNERS` file.
12+
They have permission to give more people write access to this repository.
13+
They get requested for reviews when new people add themselves to `CODEOWNERS`, and can give the relevant parties write access to the repository.
14+
15+
Every person with write access can merge a PR in the repository.
16+
The person who merges is responsible for doing so with due care.

0 commit comments

Comments
 (0)