Roles: Merge documentation infrastructure and testing infrastructure #520
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
maintainers into core infrastructure
This was referenced Mar 6, 2023
pllim
added a commit
to pllim/astropy.github.com
that referenced
this pull request
Mar 7, 2023
|
That would be fine by me |
|
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 .
|
pllim
added a commit
to pllim/astropy.github.com
that referenced
this pull request
Apr 28, 2023
|
This will be rebuild as a separate PR, where infrastrucure roles will be specified in more detail |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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-astropywithout also touchingsphinx-astropysince 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: