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 package template #523

Merged
merged 1 commit into from
Aug 17, 2023
Merged

Conversation

pllim
Copy link
Member

@pllim pllim commented Mar 6, 2023

This one might want to wait for when we officially retire the template in favor of OpenAstronomy, or should we go ahead?

Affected people in the removed listing:

@eteq
Copy link
Member

eteq commented Mar 20, 2023

I think we absolutely need a role in the project that maintains this. I think we may want to re-imagine this as more like an "OA liason" though? But I think it's important we recognize that this is a real part of the project.

@pllim
Copy link
Member Author

pllim commented May 11, 2023

I think we absolutely need a role in the project that maintains this.

@eteq , what is your concrete proposal? Remove this role, add this other role you want and state it as Unfilled ?

@pllim
Copy link
Member Author

pllim commented Aug 12, 2023

I am still of the opinion that this role should be removed until we have a concrete plan w.r.t. template coordination with OpenAstronomy and people who actually have time to do the work. I already archived package-template, so the existing description no longer applies.

@mhvk
Copy link
Contributor

mhvk commented Aug 13, 2023

I agree with @pllim that the role as described is gone, so it should be removed. I do also agree with @eteq that we should have a role like this (even if just for a transition document), but given that we have neither a role description nor a volunteer, I think that should be a separate issue -- no reason not to remove a defunct role just because we need a similar one (unless of course someone adds a description and volunteers for the role in the next day or two).

@hamogu
Copy link
Member

hamogu commented Aug 14, 2023

I agree with @mhvk : We should remove this role for now. If someone (e.g. @eteq ) then they can suggest a new role (ideally with a person to fill it) they can open a new PR.

@hamogu hamogu merged commit da55e10 into astropy:main Aug 17, 2023
1 check passed
@pllim pllim deleted the roles-rm-template branch August 17, 2023 14:05
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.

None yet

4 participants