From 312b6d3c825323ce6feba29f672b1d07c547907e Mon Sep 17 00:00:00 2001
From: Kuba Wieczorek <kuba.wieczorek@hashicorp.com>
Date: Thu, 30 Mar 2023 19:24:05 +0100
Subject: [PATCH] =?UTF-8?q?Update=20tests-completed=20job=20in=20CI=20to?=
 =?UTF-8?q?=20appear=20as=20success=20when=20one=20of=20the=E2=80=A6=20(#1?=
 =?UTF-8?q?9806)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* Update tests-completed job in CI to appear as success when one of the required checks is skipped (but not cancelled)

* Fix typo in tests-completed

---------

Co-authored-by: Marc Boudreau <marc.boudreau@hashicorp.com>
---
 .github/workflows/ci.yml | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 6a51fe619..9d2ac4a6d 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -267,6 +267,10 @@ jobs:
     - setup
     - test-go
     - test-ui
+    if: |
+      always() &&
+      !contains(needs.*.result, 'failure') &&
+      !contains(needs.*.result, 'cancelled')
     runs-on: ${{ fromJSON(needs.setup.outputs.compute-tiny) }}
     steps:
-    - run: echo "All Go test successfully passed"
+    - run: echo "All Go tests successfully passed"