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

Maintainer responsibilities #362

Merged

Conversation

minijackson
Copy link
Member

@minijackson minijackson commented Oct 22, 2022

Document maintainer responsibilities.

As discussed during the NixCon 2022 hackathon.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 22, 2022

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@minijackson
Copy link
Member Author

I have read the CLA Document and I hereby sign the CLA

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/2022-10-22-documentation-team-meeting-notes-12-nixcon-edition/22689/1

Copy link
Collaborator

@fricklerhandwerk fricklerhandwerk left a comment

Choose a reason for hiding this comment

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

Reviewed on the docs team meeting.

I would like this to be more brief, and will go over it next week. Thanks for the write-up @minijackson

maintainers/responsibilities.md Show resolved Hide resolved
maintainers/responsibilities.md Outdated Show resolved Hide resolved
maintainers/responsibilities.md Outdated Show resolved Hide resolved
maintainers/responsibilities.md Outdated Show resolved Hide resolved
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/2022-10-28-documentation-team-meeting-13/22806/1

maintainers/responsibilities.md Outdated Show resolved Hide resolved
Copy link
Member

@infinisil infinisil left a comment

Choose a reason for hiding this comment

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

We went over this today on the Nix docs meeting

maintainers/responsibilities.md Outdated Show resolved Hide resolved
maintainers/responsibilities.md Outdated Show resolved Hide resolved
maintainers/responsibilities.md Outdated Show resolved Hide resolved
maintainers/responsibilities.md Outdated Show resolved Hide resolved
maintainers/responsibilities.md Outdated Show resolved Hide resolved
maintainers/responsibilities.md Outdated Show resolved Hide resolved
maintainers/responsibilities.md Outdated Show resolved Hide resolved
maintainers/responsibilities.md Outdated Show resolved Hide resolved
Co-authored-by: Silvan Mosberger <github@infinisil.com>
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/2022-11-24-documentation-team-meeting-notes-17/23619/1

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/2022-12-01-documentation-team-meeting-notes-18/23623/1

@fricklerhandwerk
Copy link
Collaborator

fricklerhandwerk commented Dec 1, 2022

I rearranged the text slightly to provide a better reading flow, without changing the meaning. Will merge this as is. It would still be good to have an introductory paragraph. Maybe something will crystallise in the process of showing this to people.

Thank you @minijackson @infinisil @lucperkins @henrik-ch!

@fricklerhandwerk fricklerhandwerk merged commit 6c1d2a1 into NixOS:master Dec 1, 2022
fricklerhandwerk added a commit to fricklerhandwerk/nix.dev that referenced this pull request Dec 1, 2022
maintainers: document responsibilities

as discussed by documentation team

Co-authored-by: Silvan Mosberger <github@infinisil.com>
Co-authored-by: Valentin Gagarin <valentin.gagarin@tweag.io>
Co-authored-by: Emilia Bopp <contact@ebopp.de>
Co-authored-by: Luc Perkins <luc.perkins@determinate.systems>
fricklerhandwerk added a commit to fricklerhandwerk/nix.dev that referenced this pull request Dec 1, 2022
maintainers: document responsibilities

as discussed by documentation team

Co-authored-by: Silvan Mosberger <github@infinisil.com>
Co-authored-by: Valentin Gagarin <valentin.gagarin@tweag.io>
Co-authored-by: Luc Perkins <luc.perkins@determinate.systems>
Co-authored-by: Minijackson <minijackson@riseup.net>
@infinisil
Copy link
Member

@fricklerhandwerk You didn't push your changes! The merged code contains the old version 😅

@fricklerhandwerk
Copy link
Collaborator

I sure pushed something, and now I'm really confused. Will sort this out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants