conda-forge / staged-recipes

A place to submit conda recipes before they become fully fledged conda-forge feedstocks
https://conda-forge.org
BSD 3-Clause "New" or "Revised" License
701 stars 4.88k forks source link

Reviewer sub-teams #4729

Open CJ-Wright opened 6 years ago

CJ-Wright commented 6 years ago

It might be nice to have staged-recipes reviewers sub-teams. These sub-teams would feature different domain expertise. For example there could be dedicated R sub-team for reviewing R recipes. This should:

  1. Cut down on noise for reviewers (you only get pinged for stuff you know about)
  2. Help submitters get help/review from people who have the skills to help/review their recipes
  3. Still be able to ping the entire team (if you aren't certain)

Attn: @synapticarbors @conda-forge/staged-recipes @conda-forge/core

bgruening commented 6 years ago

Yes please! See https://github.com/orgs/bioconda/teams

CJ-Wright commented 6 years ago

@bgruening non-bioconda team members can't see the teams :(

patricksnape commented 6 years ago

I like the idea - would definitely make it easier to draft people in if they only had to handle platforms they were familiar with.

pelson commented 6 years ago

Yes please! 😄 I've been keen to do this from day 1 of conda-forge. There is plenty of evidence that supports doing this in terms of bringing in new long-term contributors.... https://www.youtube.com/watch?v=ZSQJYEVcMWM

What I've wanted to do is have a very clear info graphic showing the different teams and their responsibilities/expectations. Any interest in putting that together?

CJ-Wright commented 6 years ago

Who has permissions to do this?

isuruf commented 6 years ago

The core team can add new teams in conda-forge.

pelson commented 6 years ago

TBH, we could do with tidying up the way we do team in Github. Since conda-forge came along, github have added the notion of "groups of teams" which would dramatically simplify the interface that conda-forge admins see when they visit the page with 3000+ teams 😄

As for setting up the sub-teams - what are the suggestions? staged-recipes-r-lang? Any others?

synapticarbors commented 6 years ago

I think it would be helpful to have the following teams:

I think this is a fairly comprehensive list of the types of packages that I've seen, although I may be missing some.

CJ-Wright commented 6 years ago

Maybe add Julia and some python c/c++ hybrid?

jakirkham commented 6 years ago

I wonder if we can just make a group staged-recipes and add these teams below that group.

Also agree that we should probably move all feedstock teams under a group (maybe feedstocks).

CJ-Wright commented 6 years ago

Would it be possible to get an implementation of this? (even a basic version?)

igortg commented 6 years ago

Hey guys

In the last weeks, I have been jumping into some reviews on my own. Do not have much experience, but could point out some mistakes new people frequently makes (noarch, license issues, recipe yml standards, dependencies).

If you want some help, I would be willing to get into the python reviewers sub-group. Just let me know if there is some process to become an official reviewer.

rth commented 6 years ago

Since conda-forge came along, github have added the notion of "groups of teams" which would dramatically simplify the interface that conda-forge admins see when they visit the page with 3000+ teams

Another, possibly related point is that some third party CI services handle badly orgs with ~4000 repositories. For instance, I'm not part of the core dev group on conda forge, however, when I try to setup CI for some of my own packages

I'm not sure anything can be done about it during this team re-organization, but at some level I find it surprising that I'm offered to setup CI on repos I have no write access on. Isn't there anything that can be done about it on the conda-forge side? I'm not sure - maybe this was discussed somewhere else?

(This will still be an issue for conda-forge admins unfortunately :/ )

CJ-Wright commented 6 years ago
CJ-Wright commented 6 years ago

@conda-forge/staged-recipes I've started creating the language reviewer sub-teams The current format is @conda-forge/help-<language> Please see https://github.com/conda-forge/staged-recipes/issues/4729#issuecomment-374941081 for a list of the currently available sub-teams. (let me know if I need to add more).

With that said could everyone who wants to be on sub-teams please post in this issue which sub-teams they want to be on?

I will also PR into the repo a PR template which tells people about the help teams so they know who to talk to.

mwcraig commented 6 years ago

python and python/c for me please.

djsutherland commented 6 years ago

I'd do python, python/c, "some category of compiled languages", and nodejs.

synapticarbors commented 6 years ago

python and python/c

CJ-Wright commented 6 years ago

@synapticarbors do you have permissions to add yourself to the subteams? (this would make my life much simpler)

mariusvniekerk commented 6 years ago

I'd do python, python/c, maybe some java

synapticarbors commented 6 years ago

@CJ-Wright I see a button "request to join", so I'm assuming I don't have the correct level of permissions.

CJ-Wright commented 6 years ago

Ah ok I was afraid of that, I will continue adding people then.

igortg commented 6 years ago

I'd do python

CJ-Wright commented 6 years ago

@bgruening can I add you to the r team?

jakirkham commented 6 years ago

Did we add a team for go?

CJ-Wright commented 6 years ago

I have added a @conda-forge/help-go team. Is anyone interested in helping out with go?

jakirkham commented 6 years ago

^ @sodre 😉

sodre commented 6 years ago

Sign me up 👍

marcelotrevisani commented 6 years ago

Python and C/C++ for me please

tadeu commented 5 years ago

@CJ-Wright, I'm coming late to the thread, but I can help with C/C++ :)

CJ-Wright commented 5 years ago

@tadeu are you a staged-recipes reviewer?

tadeu commented 5 years ago

Not yet but I contribute with recipes/feedstocks and I'm willing to give help to recipes asking for help-c-cpp, since I have some expertise with C/C++ recipes, and saw that there are only 3 helpers in this area for now