You are viewing an old version of this page. View the current version.
Compare with Current
View Page History
Version 1
Next »
# | Item | Description | Approved Process | Notes |
---|
| PR WS Scope | PR affecting multiple Work Stream (WS) | - Not Allowed
- Allowed
- Allowed in exceptional cases
| If allowed under exceptional cases, who decides, who approves (all WSLs affected by PR?) |
| PR CH Scope | PR affecting multiple chapters | - Not Allowed
- Allowed
- Allowed in exceptional cases
|
|
| PR Topic Scope | PR affecting multiple disjointed topics | - Not Allowed
- Allowed
- Allowed in exceptional cases
|
|
| PR Comment Resolution | Who is authorised to "resolve" a comment | - Comment Author
- PR Author
- WSL if Comment Author unresponsive and WSL not PR Author
- Another Active Contributor if WSL is the PR Author
| What is the period of unresponsiveness? |
| PR Change Resolution |
| - Change Author
- Who if Change Author unresponsive
|
|
| Minimum duration a PR needs to be open for maximising input after changes complete |
| | Or should it be Minor 1 week, all others at least 2 weeks? |
| PR Re-Approvals | Ignore approvals made prior to non-typo changes | - Required After non-typo changes
- Not required
|
|
| PR Merge – non WSL author |
| |
|
| PR Merge – WSL author |
| - WSL - no additional steps
- Final Additional Reviewer to ensure all comments/changes addressed and processes observed
| Final Additional Reviewer to comment that they are OK with the PR being merged. They would have verified that the above agreed-to processes have all been observed. |