Make workflow runs wait for workflow runs (#661)

* Make workflow runs wait for workflow runs
* Split out workflows to allow standardfiles to control markdown, yaml and delivery

- Adds a "Final" workflow to set a known pass/failure hook
  - This allows us to have Terraform set the Final workflow as a required check

Signed-off-by: Dan Webb <[email protected]>

Author: damacus

.github/workflows/ci.yml

@@ -2,40 +2,13 @@
 name: ci
 
 'on':
-  pull_request:
-  push:
-    branches:
-      - main
+  workflow_run:
+    workflows: ['Lint & Unit Test']
+    types: [completed]
 
 jobs:
-  delivery:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Check out code
-        uses: actions/checkout@v2
-      - name: Run Chef Delivery
-        uses: actionshub/chef-delivery@main
-        env:
-          CHEF_LICENSE: accept-no-persist
-
-  yamllint:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Check out code
-        uses: actions/checkout@v2
-      - name: Run yaml Lint
-        uses: actionshub/yamllint@main
-
-  mdl:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Check out code
-        uses: actions/checkout@v2
-      - name: Run Markdown Lint
-        uses: actionshub/markdownlint@main
-
   integration:
-    needs: [mdl, yamllint, delivery]
+    if: ${{ github.event.workflow_run.conclusion == 'success' }}
     runs-on: ubuntu-latest
     strategy:
       matrix:
@@ -92,7 +65,7 @@ jobs:
           os: ${{ matrix.os }}
 
   integration-macos:
-    needs: [mdl, yamllint, delivery]
+    if: ${{ github.event.workflow_run.conclusion == 'success' }}
     runs-on: macos-latest
     steps:
       - name: Check out code

.github/workflows/delivery.yml

@@ -0,0 +1,34 @@
+---
+name: 'Lint & Unit Test'
+
+'on':
+  pull_request:
+  push:
+    branches: [main]
+
+jobs:
+  delivery:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v2
+      - name: Run Chef Delivery
+        uses: actionshub/chef-delivery@main
+        env:
+          CHEF_LICENSE: accept-no-persist
+
+  yamllint:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v2
+      - name: Run yaml Lint
+        uses: actionshub/yamllint@main
+
+  mdl:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v2
+      - name: Run Markdown Lint
+        uses: actionshub/markdownlint@main

.github/workflows/final.yml

@@ -0,0 +1,20 @@
+---
+name: 'Final'
+
+'on':
+  workflow_run:
+    workflows: [ci]
+    types: [completed]
+
+jobs:
+  on-success:
+    runs-on: ubuntu-latest
+    if: ${{ github.event.workflow_run.conclusion == 'success' }}
+    steps:
+      - run: exit 0
+
+  on-failure:
+    runs-on: ubuntu-latest
+    if: ${{ github.event.workflow_run.conclusion == 'failure' }}
+    steps:
+      - run: exit 1

.github/workflows/md-links.yml

@@ -1,5 +1,5 @@
 ---
-name: md-links
+name: 'Markdown Link Check'
 
 "on":
   pull_request:

CHANGELOG.md

@@ -4,6 +4,10 @@ This file is used to list changes made in each version of the Java cookbook.
 
 ## Unreleased
 
+- Split out workflows to allow standardfiles to control markdown, yaml and delivery
+- Adds a "Final" workflow to set a known pass/failure hook
+  - This allows us to have Terraform set the Final workflow as a required check
+
 ## 10.0.0 - *2021-09-02*
 
 - Remove recipes to stop confusing users