From afab80aa1ceecaeaaf635491ae90b7feaddda18f Mon Sep 17 00:00:00 2001 From: Gemma Danks <8269577+gemmadanks@users.noreply.github.com> Date: Wed, 15 Oct 2025 18:47:04 +0100 Subject: [PATCH 1/7] fix: add check for ADR to PR template --- .github/pull_request_template.md | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index b6ced17..5d733a4 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -6,6 +6,7 @@ ### Prerequisites checklist Please check each of the following before merging: - [ ] I have updated any relevant documentation +- [ ] I have created an ADR for any architecturally significant changes - [ ] I have added unit tests for any new code - [ ] I have linked to any relevant issues - [ ] I will squash my commits and, where needed, edit the commit messages to make them more informative follow the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) format From e75bdbc3df913e46f1847863b067bf1357bd0950 Mon Sep 17 00:00:00 2001 From: Gemma Danks <8269577+gemmadanks@users.noreply.github.com> Date: Wed, 15 Oct 2025 18:48:40 +0100 Subject: [PATCH 2/7] fix: simplify PR template Remove unused sections. --- .github/pull_request_template.md | 8 -------- 1 file changed, 8 deletions(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 5d733a4..a3c4f74 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -21,11 +21,3 @@ Check all that apply: - [ ] Refactor - [ ] Increasing test coverage - [ ] Other, please explain: - -### Is there anything you'd like reviewers to focus on? -- -- - -### Any other comments -- -- From ba9a01464b262160ee8a1ecd8a1d3434c7dcefe1 Mon Sep 17 00:00:00 2001 From: Gemma Danks <8269577+gemmadanks@users.noreply.github.com> Date: Wed, 15 Oct 2025 18:50:15 +0100 Subject: [PATCH 3/7] fix: reorder PR checklist --- .github/pull_request_template.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index a3c4f74..3932dbb 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -5,12 +5,12 @@ ### Prerequisites checklist Please check each of the following before merging: +- [ ] This pull request is ready to merge (leave unchecked to indicate a WIP) +- [ ] I will squash my commits and, where needed, edit the commit messages to make them more informative follow the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) format - [ ] I have updated any relevant documentation - [ ] I have created an ADR for any architecturally significant changes - [ ] I have added unit tests for any new code - [ ] I have linked to any relevant issues -- [ ] I will squash my commits and, where needed, edit the commit messages to make them more informative follow the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) format -- [ ] This pull request is ready to merge (leave unchecked to indicate a WIP) ### What is the purpose of this pull request? Check all that apply: From 7d4ce3e1e3dbca8de30494a98cdbbcc4949355f8 Mon Sep 17 00:00:00 2001 From: Gemma Danks <8269577+gemmadanks@users.noreply.github.com> Date: Wed, 15 Oct 2025 22:03:02 +0100 Subject: [PATCH 4/7] chore: update .github/pull_request_template.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- .github/pull_request_template.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 3932dbb..b49d585 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -6,7 +6,7 @@ ### Prerequisites checklist Please check each of the following before merging: - [ ] This pull request is ready to merge (leave unchecked to indicate a WIP) -- [ ] I will squash my commits and, where needed, edit the commit messages to make them more informative follow the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) format +- [ ] I have created an ADR (Architecture Decision Record) for any architecturally significant changes - [ ] I have updated any relevant documentation - [ ] I have created an ADR for any architecturally significant changes - [ ] I have added unit tests for any new code From 4b6121e562fad3d1d5ae9cbcad4c334d7971813e Mon Sep 17 00:00:00 2001 From: Gemma Danks <8269577+gemmadanks@users.noreply.github.com> Date: Wed, 15 Oct 2025 22:03:51 +0100 Subject: [PATCH 5/7] chore: update .github/pull_request_template.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- .github/pull_request_template.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index b49d585..81c8e29 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -9,7 +9,7 @@ Please check each of the following before merging: - [ ] I have created an ADR (Architecture Decision Record) for any architecturally significant changes - [ ] I have updated any relevant documentation - [ ] I have created an ADR for any architecturally significant changes -- [ ] I have added unit tests for any new code +- [ ] I will squash my commits and, where needed, edit the commit messages to make them more informative and follow the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) format - [ ] I have linked to any relevant issues ### What is the purpose of this pull request? From 5bea1b6f66e386237f1da9cfe6e776f0a2a3d2eb Mon Sep 17 00:00:00 2001 From: Gemma Danks <8269577+gemmadanks@users.noreply.github.com> Date: Wed, 15 Oct 2025 22:04:36 +0100 Subject: [PATCH 6/7] chore: update .github/pull_request_template.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- .github/pull_request_template.md | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 81c8e29..69e1d52 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -8,7 +8,6 @@ Please check each of the following before merging: - [ ] This pull request is ready to merge (leave unchecked to indicate a WIP) - [ ] I have created an ADR (Architecture Decision Record) for any architecturally significant changes - [ ] I have updated any relevant documentation -- [ ] I have created an ADR for any architecturally significant changes - [ ] I will squash my commits and, where needed, edit the commit messages to make them more informative and follow the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) format - [ ] I have linked to any relevant issues From 0a7eef7cdf5d90938af11b18ed5192dc30ea97c0 Mon Sep 17 00:00:00 2001 From: Gemma Danks <8269577+gemmadanks@users.noreply.github.com> Date: Wed, 15 Oct 2025 22:16:58 +0100 Subject: [PATCH 7/7] fix: minor edits to PR template --- .github/pull_request_template.md | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 69e1d52..07046f2 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -6,17 +6,17 @@ ### Prerequisites checklist Please check each of the following before merging: - [ ] This pull request is ready to merge (leave unchecked to indicate a WIP) -- [ ] I have created an ADR (Architecture Decision Record) for any architecturally significant changes -- [ ] I have updated any relevant documentation - [ ] I will squash my commits and, where needed, edit the commit messages to make them more informative and follow the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) format -- [ ] I have linked to any relevant issues +- [ ] I have updated relevant documentation +- [ ] I have linked to relevant issues +- [ ] I have created an ADR (Architecturural Decision Record) for architecturally significant changes ### What is the purpose of this pull request? Check all that apply: - [ ] Documentation update +- [ ] Refactor +- [ ] Improved test coverage - [ ] Bug fix -- [ ] New feature - [ ] Changes to an existing feature -- [ ] Refactor -- [ ] Increasing test coverage +- [ ] New feature - [ ] Other, please explain: