From f58851c4f85912e25be898eb795edc863e08ebdc Mon Sep 17 00:00:00 2001
From: Philip Liu <12836897+philipliu@users.noreply.github.com>
Date: Thu, 5 Oct 2023 18:12:22 -0400
Subject: [PATCH] Fix: Make Dockerhub publish dependent on npm publish (#123)

Currently, there is a race where the Dockerhub publish completes before
the npm one does, causing the image to use the older version of the test
package.

Reference: https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_run
---
 .github/workflows/docker_publish.yml | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/docker_publish.yml b/.github/workflows/docker_publish.yml
index 59eab4b..3301c9e 100644
--- a/.github/workflows/docker_publish.yml
+++ b/.github/workflows/docker_publish.yml
@@ -1,7 +1,10 @@
 name: docker image publish
 on:
-  release:
-    types: [published]
+  workflow_run:
+    workflows: ["npm publish"]
+    branches: [main]
+    types:
+      - completed
 
 jobs:
   build_and_push_docker_image: