MetOffice / MetOffice/CSET

Add some tooling for multi-person reviews

Open
#366 0 comments 0 reactions 0 assignees View on GitHub
cleanup
Dominant language
Python
Stars
33
Forks
19
Avg merge
1d 22h
Merged PRs (30d)
30

Description

While most PRs just need a single technical review, PRs adding science need to also have portability and scientific reviews. To get around GitHub only having a single setting for required reviews it would be good to add another system to work around this on a per-PR basis.

There already exist [third party actions to do this](https://github.com/EasyDynamics/pr-review-threshold), but we cannot use them due to the Met Office security policies. Instead we should make a simple action to reimplement them.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the linked pr-review-threshold action and GitHub's single required-review setting, then compare them with the repository's security-policy constraints. Done means a per-PR mechanism enforces separate portability and scientific reviews without relying on an external third-party action.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions
Domain
ci-cd
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.