Skip to content

Roles: Merge documentation infrastructure and testing infrastructure - #520

Closed
pllim wants to merge 3 commits into
astropy:mainfrom
pllim:roles-merge-doc-test
Closed

pllim wants to merge 3 commits into
astropy:mainfrom
pllim:roles-merge-doc-test

Conversation

@pllim

@pllim pllim commented Mar 6, 2023 •

Copy link
Copy Markdown
Member

Documentation and testing infrastructure are closely related and both are actually managed by the same set of people in reality. say, it is tough to maintain pytest-astropy without also touching sphinx-astropy since doctest tie them together. And both documentation and tests are done in the same manner (e.g., the CI checks for PR).

If simply merging them is too controversial, we can also try to have each of them as "sub-role" but shared by the same set of people. 🤷

Affected people in the merged listings:

TODO:

maintainers into core infrastructure
pllim added a commit to pllim/astropy.github.com that referenced this pull request Mar 7, 2023
@astrofrog

Copy link
Copy Markdown
Member

That would be fine by me

@pllim

pllim commented Mar 8, 2023

Copy link
Copy Markdown
Member Author

Given @hamogu's mention of "phone book," I could also expand this PR to add all the sub-roles for infrastructure. I mean, right now they are all the same people, but I guess it is not guaranteed to remain so when/if we finally have more staff to help out.

Please see astropy/astropy-project#118 .

astropy_infrastructure_20230307

@pllim
pllim marked this pull request as draft March 20, 2023 17:46
pllim added a commit to pllim/astropy.github.com that referenced this pull request Apr 28, 2023
@hamogu

hamogu commented May 3, 2023

Copy link
Copy Markdown
Member

This will be rebuild as a separate PR, where infrastrucure roles will be specified in more detail

@hamogu hamogu closed this May 3, 2023
@pllim
pllim deleted the roles-merge-doc-test branch May 4, 2023 04:20
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.

3 participants