This history follows the interations between four people who worked together to update two IVOA repositories.
All the interaction were via
GitHub comments and notification emails.
To update the ivoatex-docker project the Zarquan created an issue in the ivoa/ivoatex-docker repository.
https://github.com/ivoa/ivoatex-docker/issues/4
"Update to latest versions"
Zarquan created zir own fork of the ivoa/ivoatex-docker repository and worked on the code.
https://github.com/Zarquan/ivoatex-docker
Zarquan made the changes in zir fork and then created a pull request in the main ivoa/ivoatex-docker repository.
https://github.com/ivoa/ivoatex-docker/pull/6
"Updated to use Ubuntu 21.10"
Zarquan asked Sara to review the changes, which meant Sara needed to be given write access to the ivoa/ivoatex-docker repository.
Easiest way to do that was to add Sara to the ivoa/developers team.
https://github.com/orgs/ivoa/teams/developers/members
Sara completed zir review and added a comment about a problem building the SSO document.
https://github.com/ivoa/ivoatex-docker/pull/6#issuecomment-882687779
"... I have an error building the bibliography of the SSO"
Markus saw Sara's comment in an email notification and suggested a fix.
https://github.com/ivoa/ivoatex-docker/pull/6#issuecomment-883172838
"... these were legacy entries in the old bibliography which we've removed ..."
"... use 2010ivoa.rept.1123A instead of note:VOARCH and 2018ivoa.spec.0625P instead of std:VOR"
Zarquan created an issue in the ivoa-std/SSO project
https://github.com/ivoa-std/SSO/issues/2
and created zir fork of the ivoa-std/SSO repository to work on the fix.
https://github.com/Zarquan/SSO
Zarquan made the changes to fix the issue in zir own fork and then created a pull request in the ivoa-std/SSO repository.
https://github.com/ivoa-std/SSO/pull/3
"Issue 2 fix biblio"
Zarquan may have write permissions on the ivoa-std/SSO repository but ze was not able
to merge zir pull request until it had been reviewd by someone else.
Marco saw Zarquan's pull request in an email notification and approved the changes.
https://github.com/ivoa-std/SSO/pull/3#pullrequestreview-710403108
"Looks fine to me. Approved."
Once Marco had apporved the changes Zarquan was then able to merge zir changes into the ivoa-std/SSO repository.
Marco was able to see why the changes were being made because the description
of the pull request referred back to the issue in ivoa-std/SSO
https://github.com/ivoa-std/SSO/pull/3 =>
https://github.com/ivoa-std/SSO/issues/2
and the description of the issue in ivoa-std/SSO referred to Markus's comment
in the discussion about the ivoa/ivoatex-docker pull request.
https://github.com/ivoa-std/SSO/issues/2 =>
https://github.com/ivoa/ivoatex-docker/pull/6#issuecomment-883172838
Zarquan added a comment to zir pull request in the ivoa/ivoatex-docker repository
letting Sara know that the issue had been fixed in the ivoa-std/SSO repository.
https://github.com/ivoa/ivoatex-docker/pull/6#issuecomment-883341965
"@bertocco Hi Sara - the issues with the biblio codes in the SSO dcoument have been fixed. Could you re-test this for me."
Sara continued zir review and added a comment about an unnecessary step in the README file.
https://github.com/ivoa/ivoatex-docker/pull/6#issuecomment-883361807
"When you run the container using podman, youenter directly in the /document path, you don't need the cd document command mentioned in the README"
Zarquan removed the unnecessary step from the README file in zir fork, Zarquan/ivoatex-docker.
https://github.com/Zarquan/ivoatex-docker/commit/8b8a13e9b67c71f221d374c106769c83f2e5ed2f
Sara added a comment about the odd user name when run from Docker.
https://github.com/ivoa/ivoatex-docker/pull/6#issuecomment-883369640
groups: cannot find name for group ID 1000
I have no name!
Zarquan added an issue about the user name and group.
https://github.com/ivoa/ivoatex-docker/issues/9
"Fix the empty user name when run in Docker"
Sara approved the pull request and merged the changes, choosing the option to squash the chain of small commits into one step.
https://github.com/ivoa/ivoatex-docker/commit/82245585fd8e900d4abfd4bfd99df1cd4e23bceb
Four people worked on two IVOA repositories in this process, interacting via emails and comments in
GitHub.
* Sara
https://github.com/bertocco
* Markus
https://github.com/msdemlei
* Marco
https://github.com/molinaro-m
* Zarquan
https://github.com/Zarquan
*
https://github.com/ivoa/ivoatex-docker
*
https://github.com/ivoa-std/SSO
Zarquan created zir own forks of the IVOA repositories,
*
https://github.com/Zarquan/ivoatex-docker
*
https://github.com/Zarquan/SSO
but Sara, Marco and Markus only interacted with the main IVOA repositories.
Zarquan made two sets of changes, one set of changes on an issue-specific branch
https://github.com/Zarquan/SSO/tree/issue-2-fix-biblio[https://github.com/Zarquan/SSO/tree/issue-2-fix-biblio]]
and the other set of changes to the master branch of zir fork.
https://github.com/Zarquan/ivoatex-docker
Both worked, although creating an issue specific branch is probably better
if the user intends to make a number of changes to the same project.
Given that the branch is created in the users own fork, I'm not convinced we need to
require the branch names to follow a specific syntax.
The branch name does not carry over into the upstream repository when the merge is done,
so users should be free to name their branches in a way that makes sense to them.
Adding the issue number to a branch in the users fork can be odd because the
issue number refers to an issue in the main repository, not in the users own repository.
I think restricting write permissions on the main ivoa-std projects to working group chairs,
tcg members and document editors is the right level of access control.
Everyone else is free to create their own forks of the ivoa-std repositories and propose changes via pull requests.
If a group of users want to work together on a set of changes, they can create a fork in one of their
own accounts and work on the changes there.
The owner of the fork granting write access to their fork as needed.
When the group are happy with the changes, they can propose them as a single pull request to the main
ivoa-std repository for review and merging by the ivoa-std editors.
In this example Sara needed to have write permissions for the ivoa/ivoatex-docker repository
in order to be able to review and merge Zarquan's pull request.
That makes sense because Zarquan and Sara are developers working together on a software development project.
However, I don't think either Zarquan or Sara needed to have write permissions on the main ivoa-std/SSO project.
It was easy enough for Marco to step in and approved the changes in Zarquan's pull request,
and it would be an extra button click for Marco to have merged Zarquan's changes as well.
This document was written using the
ze\zir pronouns[https://www.mypronouns.org/ze-hir]].