diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 650dc5af..90c98b6f 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -9,10 +9,16 @@ We are actively encouraging users to ask questions and start discussions in the [discussions tab] of this repository. Does something seem like it's broken? Please go ahead and [open an issue]! -The [website] pages are open to contributions but they will need to be reviewed -by a member or associate member of ARC. We might be slow to approve new tool -suggestions (since we'll probably want to discuss them first) but don't let that -put you off creating an issue. +The template and [website] pages are open to contributions! But they will need +to be reviewed by a member or associate member of ARC. + +We will probably be slow to approve new tool suggestions (since we'll probably +want to discuss them first) but don't let that put you off creating an issue! + +Any opinionated changes should be approved by at least two reviewers who are +members or associate members of ARC. Pull-request authors are trusted to add two +reviewers to anything that they think might be opinionated. If a reviewer adds +a second reviewer, please wait for both to approve before merging. ## Development workflow @@ -24,8 +30,9 @@ To contribute a change, please: the tests pass if you modified the template (`pytest -s`). 3. Open a _pull request_ (PR) with changes. 4. Ask someone who did not contribute to the PR from [@UCL-ARC/collaborations] - to review it. -5. If approved with no comments, then the reviewer will merge the PR. + to review it. If it should have two reviewers, you can also request a review + from [@UCL-ARC/collaborations-python-tooling] and add the `needs-2-reviewers` label. +5. If approved with no comments, then the last approving reviewer should merge the PR. 6. If changes are requested, the PR author should address the comments, and then ask for review again. @@ -87,6 +94,7 @@ See the [jekyll docs] for more info. [Research software engineers]: https://society-rse.org/about/history [pre-commit]: https://pre-commit.com [@UCL-ARC/collaborations]: https://github.com/orgs/UCL-ARC/teams/collaborations +[@UCL-ARC/collaborations-python-tooling]: https://github.com/orgs/UCL-ARC/teams/collaborations-python-tooling [uv tool]: https://docs.astral.sh/uv/guides/tools [Install jekyll]: https://jekyllrb.com/docs/installation [jekyll docs]: https://jekyllrb.com/docs