fix(ci): handle skipped publish prerequisites

Co-authored-by: cameri <378886+cameri@users.noreply.github.com>

Author: Copilot

.github/workflows/checks.yml

@@ -217,7 +217,14 @@ jobs:
       - test-units-and-cover
       - test-integrations-and-cover
       - post-tests
-    if: github.event_name == 'push' && github.ref == 'refs/heads/main' && needs.post-tests.result == 'success'
+    if: >-
+      always() && github.event_name == 'push' && github.ref == 'refs/heads/main' &&
+      (needs.changes.result == 'success' || needs.changes.result == 'skipped') &&
+      (needs.lint.result == 'success' || needs.lint.result == 'skipped') &&
+      (needs.build-check.result == 'success' || needs.build-check.result == 'skipped') &&
+      (needs.test-units-and-cover.result == 'success' || needs.test-units-and-cover.result == 'skipped') &&
+      (needs.test-integrations-and-cover.result == 'success' || needs.test-integrations-and-cover.result == 'skipped') &&
+      needs.post-tests.result == 'success'
     env:
       REGISTRY: ghcr.io
       IMAGE_NAME: ${{ github.repository }}

test/unit/ci/publish-container-image.spec.ts

@@ -24,9 +24,11 @@ describe('container image publishing workflow', () => {
       'test-integrations-and-cover',
       'post-tests',
     ])
-    expect(publish.if).to.equal(
-      "github.event_name == 'push' && github.ref == 'refs/heads/main' && needs.post-tests.result == 'success'",
-    )
+    expect(publish.if).to.include("always() && github.event_name == 'push' && github.ref == 'refs/heads/main'")
+    for (const job of publish.needs.filter((job) => job !== 'post-tests')) {
+      expect(publish.if).to.include(`needs.${job}.result == 'success' || needs.${job}.result == 'skipped'`)
+    }
+    expect(publish.if).to.include("needs.post-tests.result == 'success'")
     expect(existsSync(join(process.cwd(), '.github', 'workflows', 'publish-container-image.yml'))).to.equal(false)
   })
 })