nixpkgs icon indicating copy to clipboard operation
nixpkgs copied to clipboard

RFC 55 - Remove inactive Nixpkgs committers

Open Mic92 opened this issue 5 years ago • 54 comments

As part of RFC55 we move inactive members of the "Nixpkgs Committers team" to the "Nixpkgs Committers Emeritus" (not created yet). The migration will happen on the 25th June.

These are the inactive accounts according to https://github.com/nix-community/rfc55:

To retain access we need at least one contribution until then.

Mic92 avatar May 25 '20 10:05 Mic92

I could not find an email address for @ts468.

Mic92 avatar May 25 '20 10:05 Mic92

I will send out emails to each affected account later this day.

Mic92 avatar May 25 '20 10:05 Mic92

@grahamc Can we move @GrahamcOfBorg to https://github.com/orgs/NixOS/teams/nixpkgs-maintainers As far as I know members of this group can also do labels and request reviews.

Mic92 avatar May 25 '20 10:05 Mic92

@Mic92 @GrahamcOfBorg needs write access to be able to push status checks.

Zimmi48 avatar May 25 '20 14:05 Zimmi48

@Mic92 @GrahamcOfBorg needs write access to be able to push status checks.

ok. I planned to ignore ofborg account anyway. It just came up as an inactive account.

Mic92 avatar May 25 '20 16:05 Mic92

@Mic92 I presume the hash.domains address listed for @ts468 in the maintainer list is not functional?

ghost avatar May 25 '20 19:05 ghost

I've removed myself from nixpgs-committers since I won't be committing any time soon, but who knows what the future will bring. Please sign me up for the emeritus newsletter & free hat.

nckx avatar May 25 '20 23:05 nckx

viric and aycanirican have pushed since, and can be removed from the list.

@Mic92 It's been a month, could you please cause the new team to be created? (My IRC setup is currently broken)

tilpner avatar Jun 28 '20 15:06 tilpner

The list is now a bit shorter:

Mic92 avatar Jul 09 '20 13:07 Mic92

Oh man, a bunch of cool folks :)

domenkozar avatar Jul 09 '20 13:07 domenkozar

They are still welcome to contribute to the project.

Mic92 avatar Jul 09 '20 13:07 Mic92

I'm still here :smile: But yes, I am "inactive" generally, and am ok with being moved.

ocharles avatar Jul 09 '20 13:07 ocharles

I apologize for the inactivity, life took a rather busy turn, and could not focus more on Nix as much as I wanted to. I am OK with being removed.

vrthra avatar Jul 09 '20 19:07 vrthra

@domenkozar @edolstra I don't have the rights to setup this group. Could you move those accounts to the right team?

Mic92 avatar Aug 25 '20 17:08 Mic92

I've created https://github.com/orgs/NixOS/teams/nixpkgs-committers-emeritus/members and invited everyone there. Going to now delete them from committers list.

domenkozar avatar Aug 26 '20 13:08 domenkozar

Done. I suggest this is automated via github actions.

domenkozar avatar Aug 26 '20 13:08 domenkozar

Since we do this every year only it would take a while until the automation pays off I suppose. I also don't know if github has API for those permission changes.

Mic92 avatar Aug 26 '20 18:08 Mic92

Once per year sounds doable, I didn't read the RFC and thought this is on a rolling basis.

domenkozar avatar Aug 28 '20 12:08 domenkozar

Hi. I have just switched back to using NixOS more frequently and will be interested in offering contributions again. Any chance I can get added back to the group with push access?

lovek323 avatar Sep 03 '20 00:09 lovek323

@lovek323 feel free to open PRs and once you have a few we can add you back :)

domenkozar avatar Sep 07 '20 14:09 domenkozar

This would be the list of inactive committers for January 2020:

Mic92 avatar Dec 17 '20 18:12 Mic92

I'd like to keep access if possible.

Fuuzetsu avatar Dec 18 '20 01:12 Fuuzetsu

If you don't mind my asking, why would you want to keep access but not be on the maintainers list? https://github.com/NixOS/nixpkgs/commit/1451a52a38f2dda459647a5c2628e7c28e17c4dc

To review and merge PRs? Get back into it when you have more time?

06kellyjac avatar Dec 18 '20 09:12 06kellyjac

In the past I have often needed some trivial software updates done and it was just much easier for a everyone involved when I was able to just push those directly and keep using the authoritative nixpkgs rather than make a fork, branch, PR, wait for review and then switch back to nixpkgs when merged.

I appreciate that anyone else could say the same so I will not be upset if I get removed here. It also would feel silly to push some random change for the sake of it though so I just asked instead.

Fuuzetsu avatar Dec 20 '20 03:12 Fuuzetsu

This is our list of inactive members as of 10th January:

Mic92 avatar Jan 10 '21 12:01 Mic92

In the past I have often needed some trivial software updates done and it was just much easier for a everyone involved when I was able to just push those directly and keep using the authoritative nixpkgs rather than make a fork, branch, PR, wait for review and then switch back to nixpkgs when merged.

I appreciate that anyone else could say the same so I will not be upset if I get removed here. It also would feel silly to push some random change for the sake of it though so I just asked instead.

If you do less than one change per year, the pull request review workflow should also work well. Most the of the longer open pull requests are non-trivial changes.

Mic92 avatar Jan 10 '21 12:01 Mic92

If you do less than one change per year, the pull request review workflow should also work well. Most the of the longer open pull requests are non-trivial changes.

It's more like 95 commits per year if you look at the commit history over time, not last year. As I said though, feel free to take me off the list.

Fuuzetsu avatar Jan 10 '21 23:01 Fuuzetsu

If you do less than one change per year, the pull request review workflow should also work well.

Especially with the recent treewide changes please at least get an ofborg eval to avoid breaking eval on master.

SuperSandro2000 avatar Feb 12 '21 00:02 SuperSandro2000

I went through the list and moved retired maintainers to their new team.

If you wish to contribute to nixpkgs, you can do so via PRs and then if it becomes frequent, ask for commit access again :)

domenkozar avatar Feb 16 '21 19:02 domenkozar

I marked this as stale due to inactivity. → More info

stale[bot] avatar Aug 17 '21 02:08 stale[bot]