From a7a85220f5da5958feb57d411e43b5dbc4269f2a Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Tue, 27 Jan 2026 09:43:36 +0000 Subject: [PATCH 01/13] CCM-13478: Initial commit with various improvements --- .github/actions/build-docs/action.yml | 4 +-- .github/actions/node-install/action.yaml | 2 ++ .github/workflows/stage-2-test.yaml | 20 +++-------- .github/workflows/stage-3-build.yaml | 34 +++++++++++++++++++ .../core-notify.component.spec.ts | 2 ++ .../pdm-poll.component.spec.ts | 2 ++ .../pdm-uploader.component.spec.ts | 2 ++ 7 files changed, 48 insertions(+), 18 deletions(-) diff --git a/.github/actions/build-docs/action.yml b/.github/actions/build-docs/action.yml index 53f74033..84430906 100644 --- a/.github/actions/build-docs/action.yml +++ b/.github/actions/build-docs/action.yml @@ -27,8 +27,8 @@ runs: uses: ruby/setup-ruby@v1.267.0 with: ruby-version: "3.4.7" # Not needed with a .ruby-version file - bundler-cache: false # runs 'bundle install' and caches installed gems automatically - #cache-version: 0 # Increment this number if you need to re-download cached gems + bundler-cache: true # Enable automatic gem caching (saves ~60-90s per build) + cache-version: 0 # Increment this number if you need to re-download cached gems working-directory: "./docs" - name: Setup Pages id: pages diff --git a/.github/actions/node-install/action.yaml b/.github/actions/node-install/action.yaml index 6ab529a5..3de09a81 100644 --- a/.github/actions/node-install/action.yaml +++ b/.github/actions/node-install/action.yaml @@ -16,6 +16,8 @@ runs: uses: actions/setup-node@v6 with: node-version: '${{ inputs.node-version }}' + cache: 'npm' + cache-dependency-path: '**/package-lock.json' - name: "Configure npm for GitHub Packages" shell: bash diff --git a/.github/workflows/stage-2-test.yaml b/.github/workflows/stage-2-test.yaml index bc01b549..4bc1dd72 100644 --- a/.github/workflows/stage-2-test.yaml +++ b/.github/workflows/stage-2-test.yaml @@ -80,6 +80,8 @@ jobs: uses: actions/setup-python@v6 with: python-version: ${{ inputs.python_version }} + cache: 'pip' + cache-dependency-path: '**/requirements*.txt' - name: "Run unit test suite" run: | make test-unit @@ -117,6 +119,8 @@ jobs: uses: actions/setup-python@v6 with: python-version: ${{ inputs.python_version }} + cache: 'pip' + cache-dependency-path: '**/requirements*.txt' - uses: ./.github/actions/node-install with: node-version: ${{ inputs.nodejs_version }} @@ -143,22 +147,6 @@ jobs: - name: "Run typecheck" run: | make test-typecheck - test-coverage: - name: "Test coverage" - needs: [test-unit] - runs-on: ubuntu-latest - timeout-minutes: 5 - permissions: - contents: read - steps: - - name: "Checkout code" - uses: actions/checkout@v5 - - name: "Run test coverage check" - run: | - make test-coverage - - name: "Save the coverage check result" - run: | - echo "Nothing to save" perform-static-analysis: name: "Perform static analysis" needs: [test-unit] diff --git a/.github/workflows/stage-3-build.yaml b/.github/workflows/stage-3-build.yaml index e9d53194..33110d16 100644 --- a/.github/workflows/stage-3-build.yaml +++ b/.github/workflows/stage-3-build.yaml @@ -33,8 +33,42 @@ on: type: string jobs: + detect-doc-changes: + name: "Detect Documentation Changes" + runs-on: ubuntu-latest + outputs: + docs_changed: ${{ steps.filter.outputs.docs }} + schemas_changed: ${{ steps.filter.outputs.schemas }} + eventcatalog_changed: ${{ steps.filter.outputs.eventcatalog }} + steps: + - name: "Checkout code" + uses: actions/checkout@v5 + + - name: "Check for relevant changes" + uses: dorny/paths-filter@v3 + id: filter + with: + filters: | + docs: + - 'docs/**' + - '**.md' + schemas: + - 'schemas/**' + - 'src/cloudevents/**' + - 'src/cloudeventjekylldocs/**' + eventcatalog: + - 'src/eventcatalog/**' + - 'src/asyncapigenerator/**' + - 'src/eventcatalogasyncapiimporter/**' + artefact-jekyll-docs: name: "Build Docs" + needs: [detect-doc-changes] + # Only build docs if relevant files changed + if: | + needs.detect-doc-changes.outputs.docs_changed == 'true' || + needs.detect-doc-changes.outputs.schemas_changed == 'true' || + needs.detect-doc-changes.outputs.eventcatalog_changed == 'true' runs-on: ubuntu-latest timeout-minutes: 15 steps: diff --git a/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts b/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts index af2d5970..ecea5221 100644 --- a/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts +++ b/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts @@ -35,6 +35,8 @@ const baseEvent: Omit = { }; test.describe('Digital Letters - Core Notify', () => { + test.describe.configure({ mode: 'parallel' }); + test.beforeAll(async () => { await purgeQueue(CORE_NOTIFIER_DLQ_NAME); test.setTimeout(250_000); diff --git a/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts b/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts index c1c413b7..269badee 100644 --- a/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts +++ b/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts @@ -41,6 +41,8 @@ const unavailableEvent = { }; test.describe('PDM Poll', () => { + test.describe.configure({ mode: 'parallel' }); + test.beforeAll(async () => { test.setTimeout(250_000); await purgeQueue(PDM_POLL_DLQ_NAME); diff --git a/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts b/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts index 5a2d99c4..067767b9 100644 --- a/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts +++ b/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts @@ -51,6 +51,8 @@ const baseEvent = { }; test.describe('Digital Letters - Upload to PDM', () => { + test.describe.configure({ mode: 'parallel' }); + test.beforeAll(async () => { await purgeQueue(PDM_UPLOADER_DLQ_NAME); test.setTimeout(250_000); From 94c4f110382d088531986e149d519cd36cd4fc8c Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Tue, 27 Jan 2026 14:05:40 +0000 Subject: [PATCH 02/13] CCM-13478: Further changes --- .github/actions/acceptance-tests/action.yaml | 4 ---- .github/workflows/stage-1-commit.yaml | 8 -------- .github/workflows/stage-2-test.yaml | 8 ++++---- .../core-notify.component.spec.ts | 2 -- .../pdm-poll.component.spec.ts | 2 -- .../pdm-uploader.component.spec.ts | 2 -- 6 files changed, 4 insertions(+), 22 deletions(-) diff --git a/.github/actions/acceptance-tests/action.yaml b/.github/actions/acceptance-tests/action.yaml index 082f687b..24b36dfc 100644 --- a/.github/actions/acceptance-tests/action.yaml +++ b/.github/actions/acceptance-tests/action.yaml @@ -34,10 +34,6 @@ runs: with: node-version: ${{ steps.nodejs_version.outputs.nodejs_version }} GITHUB_TOKEN: ${{ env.GITHUB_TOKEN }} - - name: "Repo setup" - shell: bash - run: | - npm ci - name: "Generate dependencies" shell: bash run: | diff --git a/.github/workflows/stage-1-commit.yaml b/.github/workflows/stage-1-commit.yaml index d8f3bb81..de2f3e6b 100644 --- a/.github/workflows/stage-1-commit.yaml +++ b/.github/workflows/stage-1-commit.yaml @@ -55,8 +55,6 @@ jobs: steps: - name: "Checkout code" uses: actions/checkout@v5 - with: - fetch-depth: 0 # Full history is needed to compare branches - name: "Check file format" uses: ./.github/actions/check-file-format check-markdown-format: @@ -66,8 +64,6 @@ jobs: steps: - name: "Checkout code" uses: actions/checkout@v5 - with: - fetch-depth: 0 # Full history is needed to compare branches - name: "Check Markdown format" uses: ./.github/actions/check-markdown-format terraform-docs: @@ -101,8 +97,6 @@ jobs: steps: - name: "Checkout code" uses: actions/checkout@v5 - with: - fetch-depth: 0 # Full history is needed to compare branches - name: "Check English usage" uses: ./.github/actions/check-english-usage check-todo-usage: @@ -112,8 +106,6 @@ jobs: steps: - name: "Checkout code" uses: actions/checkout@v5 - with: - fetch-depth: 0 # Full history is needed to compare branches - name: "Check TODO usage" uses: ./.github/actions/check-todo-usage detect-terraform-changes: diff --git a/.github/workflows/stage-2-test.yaml b/.github/workflows/stage-2-test.yaml index 4bc1dd72..a5ae699a 100644 --- a/.github/workflows/stage-2-test.yaml +++ b/.github/workflows/stage-2-test.yaml @@ -44,7 +44,7 @@ jobs: check-generated-dependencies: name: "Check generated dependencies" runs-on: ubuntu-latest - timeout-minutes: 5 + timeout-minutes: 4 permissions: contents: read packages: read @@ -108,7 +108,7 @@ jobs: test-lint: name: "Linting" runs-on: ubuntu-latest - timeout-minutes: 5 + timeout-minutes: 4 permissions: contents: read packages: read @@ -131,7 +131,7 @@ jobs: test-typecheck: name: "Typecheck" runs-on: ubuntu-latest - timeout-minutes: 5 + timeout-minutes: 4 permissions: contents: read packages: read @@ -154,7 +154,7 @@ jobs: permissions: id-token: write contents: read - timeout-minutes: 5 + timeout-minutes: 4 steps: - name: "Checkout code" uses: actions/checkout@v5 diff --git a/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts b/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts index ecea5221..af2d5970 100644 --- a/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts +++ b/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts @@ -35,8 +35,6 @@ const baseEvent: Omit = { }; test.describe('Digital Letters - Core Notify', () => { - test.describe.configure({ mode: 'parallel' }); - test.beforeAll(async () => { await purgeQueue(CORE_NOTIFIER_DLQ_NAME); test.setTimeout(250_000); diff --git a/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts b/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts index 269badee..c1c413b7 100644 --- a/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts +++ b/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts @@ -41,8 +41,6 @@ const unavailableEvent = { }; test.describe('PDM Poll', () => { - test.describe.configure({ mode: 'parallel' }); - test.beforeAll(async () => { test.setTimeout(250_000); await purgeQueue(PDM_POLL_DLQ_NAME); diff --git a/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts b/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts index 067767b9..5a2d99c4 100644 --- a/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts +++ b/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts @@ -51,8 +51,6 @@ const baseEvent = { }; test.describe('Digital Letters - Upload to PDM', () => { - test.describe.configure({ mode: 'parallel' }); - test.beforeAll(async () => { await purgeQueue(PDM_UPLOADER_DLQ_NAME); test.setTimeout(250_000); From e9816f6c2b3b3b537afb6ca90a50f9d6496e4715 Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Tue, 27 Jan 2026 14:33:44 +0000 Subject: [PATCH 03/13] CCM-13478: Fix npm ci --- .github/actions/acceptance-tests/action.yaml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/actions/acceptance-tests/action.yaml b/.github/actions/acceptance-tests/action.yaml index 24b36dfc..c9544d06 100644 --- a/.github/actions/acceptance-tests/action.yaml +++ b/.github/actions/acceptance-tests/action.yaml @@ -34,6 +34,10 @@ runs: with: node-version: ${{ steps.nodejs_version.outputs.nodejs_version }} GITHUB_TOKEN: ${{ env.GITHUB_TOKEN }} + - name: "Install dependencies" + shell: bash + run: | + npm ci - name: "Generate dependencies" shell: bash run: | From 25ea0084e2695cfa55a626b1d1a1b2630e26434d Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Tue, 27 Jan 2026 16:13:02 +0000 Subject: [PATCH 04/13] CCM-13478: Add read permission --- .github/workflows/stage-3-build.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/stage-3-build.yaml b/.github/workflows/stage-3-build.yaml index 33110d16..28d166d0 100644 --- a/.github/workflows/stage-3-build.yaml +++ b/.github/workflows/stage-3-build.yaml @@ -71,6 +71,8 @@ jobs: needs.detect-doc-changes.outputs.eventcatalog_changed == 'true' runs-on: ubuntu-latest timeout-minutes: 15 + permissions: + contents: read steps: - name: "Checkout code" uses: actions/checkout@v5 From 646e89bf5a55a337f06c321ff22f6c69b099d31c Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Wed, 28 Jan 2026 11:50:16 +0000 Subject: [PATCH 05/13] CCM-13478: Componenet test parallel test --- .../core-notify.component.spec.ts | 2 ++ .../digital-letters-component-tests/pdm-poll.component.spec.ts | 2 ++ .../pdm-uploader.component.spec.ts | 2 ++ 3 files changed, 6 insertions(+) diff --git a/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts b/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts index af2d5970..ecea5221 100644 --- a/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts +++ b/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts @@ -35,6 +35,8 @@ const baseEvent: Omit = { }; test.describe('Digital Letters - Core Notify', () => { + test.describe.configure({ mode: 'parallel' }); + test.beforeAll(async () => { await purgeQueue(CORE_NOTIFIER_DLQ_NAME); test.setTimeout(250_000); diff --git a/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts b/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts index c1c413b7..269badee 100644 --- a/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts +++ b/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts @@ -41,6 +41,8 @@ const unavailableEvent = { }; test.describe('PDM Poll', () => { + test.describe.configure({ mode: 'parallel' }); + test.beforeAll(async () => { test.setTimeout(250_000); await purgeQueue(PDM_POLL_DLQ_NAME); diff --git a/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts b/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts index 5a2d99c4..067767b9 100644 --- a/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts +++ b/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts @@ -51,6 +51,8 @@ const baseEvent = { }; test.describe('Digital Letters - Upload to PDM', () => { + test.describe.configure({ mode: 'parallel' }); + test.beforeAll(async () => { await purgeQueue(PDM_UPLOADER_DLQ_NAME); test.setTimeout(250_000); From 3323934c154d11f0bbc2c483c14952c373e2863b Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Wed, 28 Jan 2026 12:40:25 +0000 Subject: [PATCH 06/13] CCM-13478: Update lint/typecheck --- .github/workflows/stage-2-test.yaml | 23 +++++++++++++++++++++++ scripts/tests/lint.sh | 1 - scripts/tests/typecheck.sh | 1 - 3 files changed, 23 insertions(+), 2 deletions(-) diff --git a/.github/workflows/stage-2-test.yaml b/.github/workflows/stage-2-test.yaml index a5ae699a..608572d7 100644 --- a/.github/workflows/stage-2-test.yaml +++ b/.github/workflows/stage-2-test.yaml @@ -41,8 +41,31 @@ env: TERM: xterm-256color jobs: + detect-schema-changes: + name: "Detect Schema Changes" + runs-on: ubuntu-latest + outputs: + schemas_changed: ${{ steps.filter.outputs.schemas }} + steps: + - name: "Checkout code" + uses: actions/checkout@v5 + + - name: "Check for schema changes" + uses: dorny/paths-filter@v3 + id: filter + with: + filters: | + schemas: + - 'schemas/**' + - 'src/cloudevents/**' + - 'src/typescript-schema-generator/**' + - 'src/python-schema-generator/**' + - 'src/digital-letters-events/**' + check-generated-dependencies: name: "Check generated dependencies" + needs: [detect-schema-changes] + if: needs.detect-schema-changes.outputs.schemas_changed == 'true' runs-on: ubuntu-latest timeout-minutes: 4 permissions: diff --git a/scripts/tests/lint.sh b/scripts/tests/lint.sh index bf0435f2..f2718b69 100755 --- a/scripts/tests/lint.sh +++ b/scripts/tests/lint.sh @@ -5,5 +5,4 @@ set -euo pipefail cd "$(git rev-parse --show-toplevel)" npm ci -npm run generate-dependencies npm run lint diff --git a/scripts/tests/typecheck.sh b/scripts/tests/typecheck.sh index 292436e1..a9d50a93 100755 --- a/scripts/tests/typecheck.sh +++ b/scripts/tests/typecheck.sh @@ -5,5 +5,4 @@ set -euo pipefail cd "$(git rev-parse --show-toplevel)" npm ci -npm run generate-dependencies npm run typecheck From e5dbf2e2bedc35d1c80f01cc72a59a378a7fffca Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Wed, 28 Jan 2026 12:45:50 +0000 Subject: [PATCH 07/13] CCM-13478: Revert typecheck change --- scripts/tests/typecheck.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/scripts/tests/typecheck.sh b/scripts/tests/typecheck.sh index a9d50a93..292436e1 100755 --- a/scripts/tests/typecheck.sh +++ b/scripts/tests/typecheck.sh @@ -5,4 +5,5 @@ set -euo pipefail cd "$(git rev-parse --show-toplevel)" npm ci +npm run generate-dependencies npm run typecheck From 5beb87b54ed4413c4478bdffec76d82e94cacfea Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Wed, 28 Jan 2026 13:22:08 +0000 Subject: [PATCH 08/13] CCM-13478: Revert parallel change --- .../core-notify.component.spec.ts | 2 -- .../digital-letters-component-tests/pdm-poll.component.spec.ts | 2 -- .../pdm-uploader.component.spec.ts | 2 -- 3 files changed, 6 deletions(-) diff --git a/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts b/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts index ecea5221..af2d5970 100644 --- a/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts +++ b/tests/playwright/digital-letters-component-tests/core-notify.component.spec.ts @@ -35,8 +35,6 @@ const baseEvent: Omit = { }; test.describe('Digital Letters - Core Notify', () => { - test.describe.configure({ mode: 'parallel' }); - test.beforeAll(async () => { await purgeQueue(CORE_NOTIFIER_DLQ_NAME); test.setTimeout(250_000); diff --git a/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts b/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts index 269badee..c1c413b7 100644 --- a/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts +++ b/tests/playwright/digital-letters-component-tests/pdm-poll.component.spec.ts @@ -41,8 +41,6 @@ const unavailableEvent = { }; test.describe('PDM Poll', () => { - test.describe.configure({ mode: 'parallel' }); - test.beforeAll(async () => { test.setTimeout(250_000); await purgeQueue(PDM_POLL_DLQ_NAME); diff --git a/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts b/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts index 067767b9..5a2d99c4 100644 --- a/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts +++ b/tests/playwright/digital-letters-component-tests/pdm-uploader.component.spec.ts @@ -51,8 +51,6 @@ const baseEvent = { }; test.describe('Digital Letters - Upload to PDM', () => { - test.describe.configure({ mode: 'parallel' }); - test.beforeAll(async () => { await purgeQueue(PDM_UPLOADER_DLQ_NAME); test.setTimeout(250_000); From 331e2a8b9b6253383703bccb47a207470e16c5fb Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Wed, 28 Jan 2026 14:01:14 +0000 Subject: [PATCH 09/13] CCM-13478: Add read permission to schema check --- .github/workflows/stage-2-test.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/stage-2-test.yaml b/.github/workflows/stage-2-test.yaml index 608572d7..af510188 100644 --- a/.github/workflows/stage-2-test.yaml +++ b/.github/workflows/stage-2-test.yaml @@ -44,6 +44,8 @@ jobs: detect-schema-changes: name: "Detect Schema Changes" runs-on: ubuntu-latest + permissions: + contents: read outputs: schemas_changed: ${{ steps.filter.outputs.schemas }} steps: From df49ab885a8baa4014784ee83ab76f2b141e96e4 Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Thu, 29 Jan 2026 09:53:16 +0000 Subject: [PATCH 10/13] CCM-13478: Fix missing permission --- .github/actions/build-docs/action.yml | 2 +- .github/workflows/stage-3-build.yaml | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/.github/actions/build-docs/action.yml b/.github/actions/build-docs/action.yml index 84430906..4cc7cc53 100644 --- a/.github/actions/build-docs/action.yml +++ b/.github/actions/build-docs/action.yml @@ -27,7 +27,7 @@ runs: uses: ruby/setup-ruby@v1.267.0 with: ruby-version: "3.4.7" # Not needed with a .ruby-version file - bundler-cache: true # Enable automatic gem caching (saves ~60-90s per build) + bundler-cache: true # Enable automatic gem caching cache-version: 0 # Increment this number if you need to re-download cached gems working-directory: "./docs" - name: Setup Pages diff --git a/.github/workflows/stage-3-build.yaml b/.github/workflows/stage-3-build.yaml index 28d166d0..7bd64d9d 100644 --- a/.github/workflows/stage-3-build.yaml +++ b/.github/workflows/stage-3-build.yaml @@ -36,6 +36,8 @@ jobs: detect-doc-changes: name: "Detect Documentation Changes" runs-on: ubuntu-latest + permissions: + contents: read outputs: docs_changed: ${{ steps.filter.outputs.docs }} schemas_changed: ${{ steps.filter.outputs.schemas }} @@ -64,7 +66,6 @@ jobs: artefact-jekyll-docs: name: "Build Docs" needs: [detect-doc-changes] - # Only build docs if relevant files changed if: | needs.detect-doc-changes.outputs.docs_changed == 'true' || needs.detect-doc-changes.outputs.schemas_changed == 'true' || From 52fe2d449c0f6f79e406741dfb7e593ec4979f53 Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Fri, 30 Jan 2026 14:38:32 +0000 Subject: [PATCH 11/13] CCM-13478: Remove echo steps and increase workers --- .github/workflows/stage-2-test.yaml | 1 - .github/workflows/stage-3-build.yaml | 2 - .github/workflows/stage-4-acceptance.yaml | 44 -------------------- tests/playwright/config/playwright.config.ts | 2 +- 4 files changed, 1 insertion(+), 48 deletions(-) diff --git a/.github/workflows/stage-2-test.yaml b/.github/workflows/stage-2-test.yaml index af510188..e4a691fb 100644 --- a/.github/workflows/stage-2-test.yaml +++ b/.github/workflows/stage-2-test.yaml @@ -58,7 +58,6 @@ jobs: with: filters: | schemas: - - 'schemas/**' - 'src/cloudevents/**' - 'src/typescript-schema-generator/**' - 'src/python-schema-generator/**' diff --git a/.github/workflows/stage-3-build.yaml b/.github/workflows/stage-3-build.yaml index 7bd64d9d..223fecd1 100644 --- a/.github/workflows/stage-3-build.yaml +++ b/.github/workflows/stage-3-build.yaml @@ -53,9 +53,7 @@ jobs: filters: | docs: - 'docs/**' - - '**.md' schemas: - - 'schemas/**' - 'src/cloudevents/**' - 'src/cloudeventjekylldocs/**' eventcatalog: diff --git a/.github/workflows/stage-4-acceptance.yaml b/.github/workflows/stage-4-acceptance.yaml index f74d9211..da89bf28 100644 --- a/.github/workflows/stage-4-acceptance.yaml +++ b/.github/workflows/stage-4-acceptance.yaml @@ -22,26 +22,9 @@ on: type: string jobs: - environment-set-up: - name: "Environment set up" - runs-on: ubuntu-latest - timeout-minutes: 5 - steps: - - name: "Checkout code" - uses: actions/checkout@v5 - - name: "Create infractructure" - run: | - echo "Creating infractructure..." - - name: "Update database" - run: | - echo "Updating database..." - - name: "Deploy application" - run: | - echo "Deploying application..." test-contract: name: "Contract test" runs-on: ubuntu-latest - needs: environment-set-up timeout-minutes: 10 steps: - name: "Checkout code" @@ -55,7 +38,6 @@ jobs: test-security: name: "Security test" runs-on: ubuntu-latest - needs: environment-set-up timeout-minutes: 10 steps: - name: "Checkout code" @@ -69,7 +51,6 @@ jobs: test-ui: name: "UI test" runs-on: ubuntu-latest - needs: environment-set-up timeout-minutes: 10 steps: - name: "Checkout code" @@ -83,7 +64,6 @@ jobs: test-ui-performance: name: "UI performance test" runs-on: ubuntu-latest - needs: environment-set-up timeout-minutes: 10 steps: - name: "Checkout code" @@ -97,7 +77,6 @@ jobs: test-integration: name: "Integration test" runs-on: ubuntu-latest - needs: environment-set-up steps: - uses: actions/checkout@v5.0.0 # Calls out to the nhs-notify-internal repo. @@ -122,7 +101,6 @@ jobs: test-accessibility: name: "Accessibility test" runs-on: ubuntu-latest - needs: environment-set-up timeout-minutes: 10 steps: - name: "Checkout code" @@ -136,7 +114,6 @@ jobs: test-load: name: "Load test" runs-on: ubuntu-latest - needs: environment-set-up timeout-minutes: 10 steps: - name: "Checkout code" @@ -147,24 +124,3 @@ jobs: - name: "Save result" run: | echo "Nothing to save" - environment-tear-down: - name: "Environment tear down" - runs-on: ubuntu-latest - needs: - [ - test-accessibility, - test-contract, - test-integration, - test-load, - test-security, - test-ui-performance, - test-ui, - ] - if: always() - timeout-minutes: 5 - steps: - - name: "Checkout code" - uses: actions/checkout@v5 - - name: "Tear down environment" - run: | - echo "Tearing down environment..." diff --git a/tests/playwright/config/playwright.config.ts b/tests/playwright/config/playwright.config.ts index 9d85ebcb..d92c4c6e 100644 --- a/tests/playwright/config/playwright.config.ts +++ b/tests/playwright/config/playwright.config.ts @@ -8,7 +8,7 @@ export default defineConfig({ forbidOnly: !!process.env.CI, retries: 0, /* Opt out of parallel tests on CI. */ - workers: process.env.CI ? 8 : undefined, + workers: process.env.CI ? 20 : undefined, /* Reporter to use. See https://playwright.dev/docs/test-reporters */ reporter: [ ['line'], From 3839e572f1d8c149dc96e4e264e242fe4da54bf1 Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Fri, 30 Jan 2026 15:17:38 +0000 Subject: [PATCH 12/13] CCM-13478: Increase workers to 35 --- tests/playwright/config/playwright.config.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/playwright/config/playwright.config.ts b/tests/playwright/config/playwright.config.ts index d92c4c6e..47773b21 100644 --- a/tests/playwright/config/playwright.config.ts +++ b/tests/playwright/config/playwright.config.ts @@ -8,7 +8,7 @@ export default defineConfig({ forbidOnly: !!process.env.CI, retries: 0, /* Opt out of parallel tests on CI. */ - workers: process.env.CI ? 20 : undefined, + workers: process.env.CI ? 35 : undefined, /* Reporter to use. See https://playwright.dev/docs/test-reporters */ reporter: [ ['line'], From d397efc3d788b24ef47bad55a10e69e1c89ed714 Mon Sep 17 00:00:00 2001 From: "scott.fullerton1" Date: Fri, 30 Jan 2026 15:46:21 +0000 Subject: [PATCH 13/13] CCM-13478: Reset workers to 8 --- tests/playwright/config/playwright.config.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/playwright/config/playwright.config.ts b/tests/playwright/config/playwright.config.ts index 47773b21..9d85ebcb 100644 --- a/tests/playwright/config/playwright.config.ts +++ b/tests/playwright/config/playwright.config.ts @@ -8,7 +8,7 @@ export default defineConfig({ forbidOnly: !!process.env.CI, retries: 0, /* Opt out of parallel tests on CI. */ - workers: process.env.CI ? 35 : undefined, + workers: process.env.CI ? 8 : undefined, /* Reporter to use. See https://playwright.dev/docs/test-reporters */ reporter: [ ['line'],