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

Roles: Remove sub-role for Learn Team #525

Closed
wants to merge 2 commits into from

Conversation

pllim
Copy link
Member

@pllim pllim commented Mar 6, 2023

I think it is a little overkill to have so many sub-roles for Learn Team. Infrastructure staff maintains multiple packages across the board (and manages releases for them too, etc) and yet we do not list all those packages on this page.

This PR eliminates sub-role for Learn Team but kept them in parenthesis since I also notice you have sub-responsibility. This is not ideal but it is getting late and I cannot think of anything better. Ideas welcome. If we can make this more fluid, then when you move Learn staff around the team, you don't have to constantly come back to update this page.

Affected people in this listing:

TODO (from #524 (comment)):

  • learn/README.md to capture info removed from Roles page and move "the description at the bottom of the page" over
  • Link to learn/ from roles

Depends on:

Also see:

@jonathansick
Copy link

This seems reasonable to me if it's easier to maintain the team page without all the sub-roles 👍

The fine-grained subroles might even be better maintained in other ways. e.g., a GitHub team for content editors could then let us assign PR reviews automatically.

@hamogu
Copy link
Member

hamogu commented Mar 7, 2023

The learn team seems to coordinate within themselves with regular telecons. Also, I think several people are stepping back from active engagement, so maybe this is time to restructure it and summary a bit.

@pllim
Copy link
Member Author

pllim commented Mar 7, 2023

@jonathansick , is astropy/learn-astropy#71 sufficient for follow-up? I get confused now between astropy-tutorials and learn-astropy. Is this the correct repo?

EDIT: I also opened astropy/astropy-tutorials#580 just in case.

pllim added a commit to pllim/astropy-project that referenced this pull request Mar 7, 2023
@pllim
Copy link
Member Author

pllim commented Mar 7, 2023

@kelle , I don't know how I feel about the duplicated info but I opened astropy/astropy-project#332 to address your request. As long as we remember to keep them in sync...

@jonathansick
Copy link

@jonathansick , is astropy/learn-astropy#71 sufficient for follow-up? I get confused now between astropy-tutorials and learn-astropy. Is this the correct repo?

@pllim Thanks for making those issues! astropy/astropy-tutorials is content-oriented and astropy/learn-astropy is infrastructure-oriented so that's a good expression of the editorial vs infrastructure sub-roles for example.

Copy link
Member

@eteq eteq left a comment

Choose a reason for hiding this comment

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

See my inline comment about workshop role. But also I feel like the discussion #524 applies here - that there is value in ensuring we don't lose the information of all the work. Should we maybe just copy over the info here?

}
]
},
Copy link
Member

Choose a reason for hiding this comment

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

I think the workshop coordinator work is really a distinct role that aligns with Learn but should be highlighted as a quite different set of work. Obviously it's part of the Learn effort, but I think we lose a lot of information by not recording this. After all one of the main points of the roles page is to know "who is doing this work, if anyone", and this is critical work.

Suggested change
},
},
{
"role": "Workshop Coordinator",
"url": "Workshop_coordinator",
"people": [
"David Shupe",
"Brett Morris",
"Kelle Cruz"
],
"role-head": "Workshop Coordinator",
"responsibilities": {
"description": "Organize and coordinate Astropy workshops for training and outreach to users",
"details": [
"Maintain the astropy-workshops repository",
"Oversee staffing/volunteers for workshops",
"Identify opportunities for workshops in diverse geographic locations"
]
}
},

@eteq
Copy link
Member

eteq commented Mar 20, 2023

(I had to do something a bit weird in the suggestion there, so @pllim , if you accept that you'll definitely want to wait for the CI to be sure I didn't break teh JSON)

@hamogu
Copy link
Member

hamogu commented May 3, 2023

After discussion in the Coordination meeting, we believe that it's worthwhile to have relatively fine-grained roles, even if they might be unfilled. Names should be updated (or be replaced by "unfilled") in a separate PR.

@hamogu hamogu closed this May 3, 2023
@pllim pllim deleted the roles-rm-learn-subroles branch May 4, 2023 04:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants