scikit-learn-extra icon indicating copy to clipboard operation
scikit-learn-extra copied to clipboard

RFC Process for giving commit rights

Open rth opened this issue 5 years ago • 12 comments

I think we should formalize a bit way commit rights are given for scikit-learn-extra.

Currently following groups can modify code in this repo,

  • @scikit-learn-contrib/scikit-learn-extra team (currently containing @glemaitre and myself)¹
  • owners of the scikit-learn-contrib org (28 people in total)

I think this is not very sustainable and we should give commit rights to scikit-learn-extra for contributors who have made significant contributions (in code or otherwise cf https://scikit-learn.org/stable/developers/contributing.html) and are interested/available in helping with the maintenance work.

(Also I guess the use of groups for managing permissions in this org could be improved in general).

rth avatar Jul 29 '19 13:07 rth

I think this is not very sustainable and we should give commit rights to scikit-learn-extra for contributors who have made significant contributions (in code or otherwise cf https://scikit-learn.org/stable/developers/contributing.html) and are interested/available in helping with the maintenance work.

I am +1

glemaitre avatar Jul 30 '19 07:07 glemaitre

yeah happy to expand the committer base if you have an idea who to add.

amueller avatar Jul 30 '19 16:07 amueller

I would like to give commit rights for this repo to @chkoar who helped with code review and is already co-maintaining imbalanced-learn.

rth avatar Feb 25 '20 20:02 rth

+1

glemaitre avatar Feb 25 '20 21:02 glemaitre

+1

GaelVaroquaux avatar Feb 25 '20 21:02 GaelVaroquaux

+1

adrinjalali avatar Feb 26 '20 10:02 adrinjalali

no objection

agramfort avatar Feb 26 '20 11:02 agramfort

It's done :)

rth avatar Feb 26 '20 11:02 rth

+1

chkoar avatar Feb 26 '20 12:02 chkoar

I would like to propose to add https://github.com/TimotheeMathieu to the scikit-learn-extra repo team, who has been doing useful contributions to this library for the last several months.

rth avatar Nov 09 '20 11:11 rth

Fine for me! Thanks for suggesting

GaelVaroquaux avatar Nov 10 '20 09:11 GaelVaroquaux

It's done.

rth avatar Nov 11 '20 07:11 rth