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

Update team page with council membership list #569

Merged
merged 9 commits into from
Nov 29, 2022

Conversation

minrk
Copy link
Member

@minrk minrk commented Sep 30, 2022

closes #566

I added everyone who said at least 'maybe' for the Council, since we don't have too many (9), and folks can always step down.

Adds the team membership to each team member card. However, all currently active team members are steering council members. I added some more todos in #497 (comment) because we haven't updated the other team lists, which still need updating. Including recognizing several folks who I think ought to be current 'team members' by the new definition but aren't listed right now.

Copy link
Member

@choldgraf choldgraf left a comment

Choose a reason for hiding this comment

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

This looks good to me - it is a nice refresh to reflect the current team dynamics. I think there might be more stuff to improve as we learn about these roles more but IMO it's good to merge.

I'm not sure how to interpret the fact that everybody is on the council. As the group size is still within our stated target size, my feeling is that it's OK. But I'm wondering what circumstances would lead somebody to self-identify as maintainer instead of council? Maybe it is just most-appropriate for newer members of the team?

" ",
)

# team_titles = {
Copy link
Member

Choose a reason for hiding this comment

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

If you'd like to do something with these sections later on, could you leave a # TODO comment to explain why they're commented out?

Copy link
Member Author

Choose a reason for hiding this comment

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

Added a note about what this was for - we could have a list like the 'inactive members' list for each team. Or we could list teams on the cards. Either way!

@minrk
Copy link
Member Author

minrk commented Oct 1, 2022

As I tried to describe in #497 (comment), I think there are several more folks that should be listed as team members / maintainers, who aren't listed yet, since we had before was mainly the one 'core team' which maps most directly to the council. Now that we have more distinct roles, it makes sense for more folks to be listed here.

@choldgraf
Copy link
Member

Ah yes, i missed that comment. I think that makes a lot of sense. Also good point that being a maintainer is usually scoped to one or more repositories, whereas steerco is more like the whole project.

Maybe we should generate a list of each repository in the org with the listed maintainers for it. That way it will also be clear if a repository does not have a maintainer.

@choldgraf
Copy link
Member

Just to be clear, my comment above is not blocking, I think we should merge this and iterate

Co-authored-by: Simon Li <orpheus+devel@gmail.com>
@choldgraf
Copy link
Member

Anything blocking this from moving forward?

@afshin
Copy link

afshin commented Nov 15, 2022

pinging: @damianavila @ivanov @fperez

Damian, Paul, and Fernando are members of the original SC who requested to be members of the JupyterHub Council as we transition away from the SC. Please can we add them in this PR?

@minrk
Copy link
Member Author

minrk commented Nov 15, 2022

Sure! If @damianavila @ivanov @fperez could add

  • affiliation
  • focus
  • contributions

like what other folks have (you can see examples in the pr) in a comment, I'll add you to the PR.

(that will also be a record of their intent, since I don't seem to have that)

@fperez
Copy link
Collaborator

fperez commented Nov 16, 2022

Thanks for the ping, done!.

consideRatio and others added 3 commits November 16, 2022 09:30
Co-authored-by: Tim Head <betatim@gmail.com>
@damianavila
Copy link
Contributor

Will do it tomorrow, thanks for the ping!

@choldgraf
Copy link
Member

I suggest that we merge this one in. If there are small adjustments to make, we can always do this in a subsequent PR, but it is an iterative improvement and will reflect the reality of our current team better than the docs currently do.

Are there any objections to this idea?

@minrk
Copy link
Member Author

minrk commented Nov 28, 2022

@choldgraf I think it's all set!

@consideRatio
Copy link
Member

There seem to be agreement that this should be merged so I'll go for it, it looks fine to me as well!

@consideRatio consideRatio merged commit 7386442 into jupyterhub:main Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refreshing council membership
9 participants