Merge pull request #6921 from nextcloud/enh/noid/workflow-summaries

Add summary jobs to workflows
nextcloud · Sep 9, 2022 · 1c2a5d0 · 1c2a5d0
2 parents 665cc85 + e28066a
commit 1c2a5d0
Show file tree

Hide file tree

Showing 3 changed files with 54 additions and 0 deletions.
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -74,3 +74,25 @@ jobs:
       run: npm run lint
     env:
       CI: true
+
+  summary:
+    runs-on: ubuntu-latest
+    needs:
+      - xml-linters
+      - php-linters
+      - php-cs-fixer
+      - node-linters
+
+    if: always()
+
+    name: lint-summary
+
+    steps:
+      - name: XML linter status
+        run: if ${{ needs.xml-linters.result != 'success' && needs.xml-linters.result != 'skipped' }}; then exit 1; fi
+      - name: PHP linter status
+        run: if ${{ needs.php-linters.result != 'success' && needs.php-linters.result != 'skipped' }}; then exit 1; fi
+      - name: PHP cs fixer status
+        run: if ${{ needs.php-cs-fixer.result != 'success' && needs.php-cs-fixer.result != 'skipped' }}; then exit 1; fi
+      - name: Node linter status
+        run: if ${{ needs.node-linters.result != 'success' && needs.node-linters.result != 'skipped' }}; then exit 1; fi
diff --git a/.github/workflows/static-analysis.yml b/.github/workflows/static-analysis.yml
@@ -31,3 +31,16 @@ jobs:
             run: composer require --dev christophwurst/nextcloud:${{ matrix.ocp-version }} --ignore-platform-reqs
           - name: Run coding standards check
             run: composer run psalm
+
+  summary:
+    runs-on: ubuntu-latest
+    needs:
+      - static-psalm-analysis
+
+    if: always()
+
+    name: static-analysis-summary
+
+    steps:
+      - name: Static analysis status
+        run: if ${{ needs.static-psalm-analysis.result != 'success' && needs.static-psalm-analysis.result != 'skipped' }}; then exit 1; fi
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -203,3 +203,22 @@ jobs:
             run: npm run test:unit
       env:
           CI: true
+
+  summary:
+    runs-on: ubuntu-latest
+    needs:
+      - unit-tests
+      - integration-tests
+      - frontend-unit-test
+
+    if: always()
+
+    name: test-summary
+
+    steps:
+      - name: Unit test status
+        run: if ${{ needs.unit-tests.result != 'success' && needs.unit-tests.result != 'skipped' }}; then exit 1; fi
+      - name: Integration test status
+        run: if ${{ needs.integration-tests.result != 'success' && needs.integration-tests.result != 'skipped' }}; then exit 1; fi
+      - name: Frontend unit test status
+        run: if ${{ needs.frontend-unit-test.result != 'success' && needs.frontend-unit-test.result != 'skipped' }}; then exit 1; fi