From 505131b01c0c4ec22aa8ae36aa8683f204e6b08e Mon Sep 17 00:00:00 2001 From: l Date: Tue, 7 Jul 2026 14:14:55 +0100 Subject: [PATCH 1/5] allow pr_target test runs --- .github/workflows/validate-pr-metadata.yml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/validate-pr-metadata.yml b/.github/workflows/validate-pr-metadata.yml index 627807841..c4e1d99e3 100644 --- a/.github/workflows/validate-pr-metadata.yml +++ b/.github/workflows/validate-pr-metadata.yml @@ -24,7 +24,10 @@ jobs: if: ${{ github.event.label.name == 'Needs Review' }} steps: - name: checkout base branch - uses: actions/checkout@v5 + uses: actions/checkout@v7 + allow-unsafe-pr-checkout: true + with: + ref: ${{ github.event.pull_request.head.sha }} - name: Get changed files id: changed-files uses: tj-actions/changed-files@v47.0.5 From 3675c148949c037931232c1a2dcdf91b4971aa91 Mon Sep 17 00:00:00 2001 From: l Date: Tue, 7 Jul 2026 14:28:28 +0100 Subject: [PATCH 2/5] fix action input --- .github/workflows/validate-pr-metadata.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/validate-pr-metadata.yml b/.github/workflows/validate-pr-metadata.yml index c4e1d99e3..9b0d7c6cc 100644 --- a/.github/workflows/validate-pr-metadata.yml +++ b/.github/workflows/validate-pr-metadata.yml @@ -25,8 +25,8 @@ jobs: steps: - name: checkout base branch uses: actions/checkout@v7 - allow-unsafe-pr-checkout: true with: + allow-unsafe-pr-checkout: true ref: ${{ github.event.pull_request.head.sha }} - name: Get changed files id: changed-files From a49c91e3a96ef72a89ed9626d9d08b76a37b2da1 Mon Sep 17 00:00:00 2001 From: l Date: Tue, 7 Jul 2026 14:50:51 +0100 Subject: [PATCH 3/5] Fix test case sed/4 --- expect/individual-shell-tools/sed/script-04.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/expect/individual-shell-tools/sed/script-04.sh b/expect/individual-shell-tools/sed/script-04.sh index 852fa1557..8bfc58ad8 100755 --- a/expect/individual-shell-tools/sed/script-04.sh +++ b/expect/individual-shell-tools/sed/script-04.sh @@ -2,7 +2,7 @@ This is a sample file for experimenting with sed. It contains many lines, and there are some things you may want to do with each of them. -We'll include some score information: +We will include some score information: 37 Alisha 15 Jacob 7 Pietro From 0b0eefbb3c63f2f504c0da8ae6bc785c32239b1c Mon Sep 17 00:00:00 2001 From: l Date: Mon, 13 Jul 2026 10:09:49 +0100 Subject: [PATCH 4/5] fix test case for jq 3 --- expect/jq/script-03.sh | 1 - 1 file changed, 1 deletion(-) diff --git a/expect/jq/script-03.sh b/expect/jq/script-03.sh index d63f6ec74..87334dd44 100755 --- a/expect/jq/script-03.sh +++ b/expect/jq/script-03.sh @@ -1,2 +1 @@ Selma, Software Engineer -Selma, Software Engineer From bb71b06c43ff4c7ecc2a78e4df1a7d07a73c6605 Mon Sep 17 00:00:00 2001 From: LonMcGregor <3817332+LonMcGregor@users.noreply.github.com> Date: Wed, 12 Aug 2026 15:05:13 +0100 Subject: [PATCH 5/5] Add helper message for tests (#648) The way the tests work, it requires removing and re-adding the label. I have seen a few trainees confused about this. This adds a little note about it at the end. --------- Co-authored-by: l --- test-sdc.sh | 2 ++ 1 file changed, 2 insertions(+) diff --git a/test-sdc.sh b/test-sdc.sh index 62d1c7e1f..432993cf3 100755 --- a/test-sdc.sh +++ b/test-sdc.sh @@ -180,3 +180,5 @@ elif [[ "$1" == "implement-cowsay" ]]; then else echo "Please run this with a valid test directory name as argument" fi + +echo "If you want to re-run the automated tests after changing some code, remove the 'Needs Review' label, wait a little, and then re-add the 'Needs Review' label on your PR." >> testoutput.txt