Skip to content

Comments

Add myself as a code owner of the contributing documentation#248166

Merged
infinisil merged 2 commits intoNixOS:masterfrom
infinisil:contributing-docs-codeowner
Aug 9, 2023
Merged

Add myself as a code owner of the contributing documentation#248166
infinisil merged 2 commits intoNixOS:masterfrom
infinisil:contributing-docs-codeowner

Conversation

@infinisil
Copy link
Member

I want to take ownership of this part of the documentation, especially with the cleanup in #245243. Doing so before that PR is merged also allows me to get notified about any potential future merge conflicts before they happen.

I want to take ownership of this part of the documentation, especially
with the cleanup in NixOS#245243. Doing so
before that PR is merged also allows me to get notified about any potential
future merge conflicts before they happen.
@infinisil infinisil requested a review from edolstra as a code owner August 9, 2023 18:01
@github-actions github-actions bot added the 6.topic: policy discussion Discuss policies to work in and around Nixpkgs label Aug 9, 2023
@infinisil infinisil changed the title CODEOWNERS: Add myself to the contributing documentation Add myself as a code owner of the contributing documentation Aug 9, 2023
Copy link
Member

@Janik-Haag Janik-Haag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds like a good idea.

/doc/contributing/contributing-to-documentation.chapter.md @jtojnar @fricklerhandwerk
# Contributor documentation
/CONTRIBUTING.md @infinisil
/.github/PULL_REQUEST_TEMPLATE.md
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be /.github/PULL_REQUEST_TEMPLATE.md @infinisil?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh damn yeah, sorry I merged it already, but I'll open a new one for that!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Argh sorry, I should have been more careful. Also why does github say the file is valid with a basically empty entry?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it's not technically invalid, it just doesn't make much sense, I wouldn't mind them adding a check for that. It does have a slight effect though because codeowners of subdirectories override code owners of the parents.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because it actually is valid: A line with no owner is used to exclude a path from ownership.

# In this example, @octocat owns any file in the `/apps`
# directory in the root of your repository except for the `/apps/github`
# subdirectory, as its owners are left empty.
/apps/ @octocat
/apps/github

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in #248168

@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Aug 9, 2023
@infinisil
Copy link
Member Author

No need to wait for CI, and GitHub confirmed to me that

This CODEOWNERS file is valid.

@infinisil infinisil merged commit 0ab34b2 into NixOS:master Aug 9, 2023
@infinisil infinisil deleted the contributing-docs-codeowner branch August 9, 2023 18:21
infinisil added a commit that referenced this pull request Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: policy discussion Discuss policies to work in and around Nixpkgs 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants