diff options
author | Richard Steinmetz <richard@steinmetz.cloud> | 2022-09-09 12:18:12 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-09-09 12:18:12 +0300 |
commit | 1c2a5d01d37e6f72f19d41fc48e06b88495a1bfd (patch) | |
tree | dc694af965f1a8ec04135d2f301c787962131859 | |
parent | 665cc85f2dd7bd86385954a1070e57139692c10f (diff) | |
parent | e28066a7dc9a8b3053369ca62c455738860c0ecb (diff) |
Merge pull request #6921 from nextcloud/enh/noid/workflow-summaries
Add summary jobs to workflows
-rw-r--r-- | .github/workflows/lint.yml | 22 | ||||
-rw-r--r-- | .github/workflows/static-analysis.yml | 13 | ||||
-rw-r--r-- | .github/workflows/test.yml | 19 |
3 files changed, 54 insertions, 0 deletions
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml index 510442c37..27d588b95 100644 --- 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 index fe2f141e4..89be11299 100644 --- 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 index 8c7352059..1308d0330 100644 --- 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 |