Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CODEOWNERS: Add Profpatsch to /lib/generators.nix #37694

Merged
merged 1 commit into from Mar 24, 2018
Merged

Conversation

Profpatsch
Copy link
Member

I’d like to be notified of changes so I can ensure the API always follows the same patterns.

Later (more specific) files replace more general codeowners according to the documentation, so I also listed edolstra and nbp again.

I’d like to be notified of changes so I can ensure the API always follows the same patterns.

Later (more specific) files replace more general codeowners [according to the documentation](https://help.github.com/articles/about-codeowners/), so I also listed `edolstra` and `nbp` again.
Copy link
Member

@FRidh FRidh left a comment

Choose a reason for hiding this comment

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

I am against this change. The CODEOWNERS file has one clear purpose: to describe who owns what. We should not misuse it just to receive notifications. If one wants notifications, we can look into another mechanism (like running our own instance of the mention-bot).

@Profpatsch
Copy link
Member Author

Profpatsch commented Mar 23, 2018

The CODEOWNERS file has one clear purpose: to describe who owns what.

Well, since I wrote that module and am the only contributor so far (apart from a few minor treewide changes), I do consider myself the “owner”, insofar as I have an substantial interest in reviewing proposed changes to the file.
That is perhaps the same reason your nick is bound to the python modules?

@FRidh
Copy link
Member

FRidh commented Mar 24, 2018

I do consider myself the “owner”

That was not obvious from the PR.

That is perhaps the same reason your nick is bound to the python modules?

I'm as maintainer not so much interested in the actual modules, that is up to the module maintainers (when they have one), but I am, in the structure.

@Profpatsch
Copy link
Member Author

That was not obvious from the PR.

Yes, I should have clarified that.

@nbp nbp merged commit 99f2b33 into master Mar 24, 2018
@primeos primeos deleted the generators-codeowners branch April 6, 2018 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants