From 085ef82df4375255cc4fcdd17f1e1af10dd411fa Mon Sep 17 00:00:00 2001
From: Sidney Richards <sidney@maykinmedia.nl>
Date: Thu, 19 Dec 2024 14:03:11 +0100
Subject: [PATCH] fixup! Remove docs from CI dependencies for docker push

---
 .github/workflows/ci.yml | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index d839a335..8d1f806a 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -8,6 +8,12 @@ on:
       - '**'
   pull_request:
   workflow_dispatch:
+  workflow_run:
+    workflows:
+      - Check requirements files
+      - Docker image build
+    types:
+      - completed
 
 env:
   IMAGE_NAME: maykinmedia/objects-api
@@ -95,15 +101,10 @@ jobs:
       docker-image-name: ${{ needs.store-reusable-workflow-vars.outputs.image-name }}
 
   open-api-publish:
+    if: ${{ github.event.workflow_run.conclusion == 'success' }}
     uses: maykinmedia/open-api-workflows/.github/workflows/publish.yml@v1
     needs:
       - store-reusable-workflow-vars
-      # TODO: We explicitly specify the jobs to depend on here, to avoid depending on
-      # docs, which will be skipped by default (thus not fulfilling the dependency for
-      # the push), but can't be enabled because the docs build is currently broken.
-      - open-api-ci/check-requirements
-      - open-api-ci/docker-build
-      - open-api-ci/image-scan
       - tests
     with:
       docker-image-name: ${{ needs.store-reusable-workflow-vars.outputs.image-name }}