Skip to content

Commit 0ed7a67

Browse files
committed
CODEOWNERS: add @NixOS/steering as the fallback codeowner of every documentation file
The situation of #217 with no merge happening because of incorrect paperwork -- even though it's just documenting a change that already happened -- is *ridiculous*. Let's fix that.
1 parent fa2bd9f commit 0ed7a67

2 files changed

Lines changed: 25 additions & 21 deletions

File tree

.github/CODEOWNERS

Lines changed: 22 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,35 +1,36 @@
11
# See doc/org-repo.md
2+
/ @NixOS/steering
23

3-
/README.md @NixOS/org
4-
/CONTRIBUTING.md @NixOS/org
5-
/LICENSE @NixOS/org
4+
/README.md @NixOS/steering @NixOS/org
5+
/CONTRIBUTING.md @NixOS/steering @NixOS/org
6+
/LICENSE @NixOS/steering @NixOS/org
67

7-
/.gitignore @NixOS/org
8-
/.github/CODEOWNERS @NixOS/org
9-
/.github/workflows @NixOS/org
10-
/scripts @NixOS/org
11-
/npins @NixOS/org
12-
/default.nix @NixOS/org
13-
/shell.nix @NixOS/org
14-
/rulesets @NixOS/org
15-
/rulesets/nixpkgs @NixOS/nixpkgs-core
8+
/.gitignore @NixOS/steering @NixOS/org
9+
/.github/CODEOWNERS @NixOS/steering @NixOS/org
10+
/.github/workflows @NixOS/steering @NixOS/org
11+
/scripts @NixOS/steering @NixOS/org
12+
/npins @NixOS/steering @NixOS/org
13+
/default.nix @NixOS/steering @NixOS/org
14+
/shell.nix @NixOS/steering @NixOS/org
15+
/rulesets @NixOS/steering @NixOS/org
16+
/rulesets/nixpkgs @NixOS/steering @NixOS/nixpkgs-core
1617

1718
/doc/org-repo.md @NixOS/steering
1819
/doc/discourse.md @NixOS/steering @mweinelt @lassulus @jtojnar @ctheune @dpausp
19-
/doc/github.md @NixOS/org
20+
/doc/github.md @NixOS/steering @NixOS/org
2021
/doc/github-org-owners.md @NixOS/steering
2122
/doc/matrix.md @NixOS/steering
2223
/doc/moderation.md @NixOS/steering
23-
/doc/nixos-releases.md @NixOS/nixpkgs-core
24+
/doc/nixos-releases.md @NixOS/steering @NixOS/nixpkgs-core
2425
/doc/resources.md @NixOS/steering
25-
/doc/mail.md @NixOS/infra
26-
/doc/freescout.md @infinisil
26+
/doc/mail.md @NixOS/steering @NixOS/infra
27+
/doc/freescout.md @NixOS/steering @infinisil
2728
/doc/governance.md @NixOS/steering
2829
/doc/values.md @NixOS/steering
29-
/doc/vault.md @Mic92 @mweinelt @zimbatm
30+
/doc/vault.md @NixOS/steering @Mic92 @mweinelt @zimbatm
3031
/doc/constitution.md @NixOS/steering
31-
/doc/nixpkgs-core.md @NixOS/nixpkgs-core
32-
/doc/nixpkgs-committers.md @NixOS/nixpkgs-core
33-
/doc/calendar.md @edolstra @tomberek @fricklerhandwerk
32+
/doc/nixpkgs-core.md @NixOS/steering @NixOS/nixpkgs-core
33+
/doc/nixpkgs-committers.md @NixOS/steering @NixOS/nixpkgs-core
34+
/doc/calendar.md @NixOS/steering @edolstra @tomberek @fricklerhandwerk
3435
/doc/nixcon.md @NixOS/steering
35-
/doc/hetzner.md @NixOS/infra
36+
/doc/hetzner.md @NixOS/steering @NixOS/infra

doc/org-repo.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,3 +8,6 @@ PRs can only be merged if a codeowner for the respective files approves it, and
88

99
Furthermore, the code owners for the CODEOWNERS file should have permission to give more people write access to this repository.
1010
These people get requested for reviews when new people add themselves to CODEOWNERS, allowing them to give write access when merged.
11+
12+
Every member of the steering committee has the right to merge a PR in the repository.
13+
The person who merges is responsible for doing so with due care.

0 commit comments

Comments
 (0)