Skip to content

Commit 53f831b

Browse files
YooSunYoungnvaytet
andauthored
Add report job status step to CI workflow (#172)
* Add report job status step to CI workflow * Comment out package check in CI workflow Commented out the condition checking for empty packages in test and docs jobs. * Update CI workflow conditions for tests and docs * Refactor CI job status reporting logic * Remove white spaces --------- Co-authored-by: Neil Vaytet <39047984+nvaytet@users.noreply.github.com>
1 parent 5c7d679 commit 53f831b

1 file changed

Lines changed: 21 additions & 0 deletions

File tree

.github/workflows/ci.yml

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -110,3 +110,24 @@ jobs:
110110
name: docs-${{ matrix.package }}
111111
path: packages/${{ matrix.package }}/html
112112
- run: echo "::notice::https://remote-unzip.deno.dev/${{ github.repository }}/artifacts/${{ steps.artifact-upload-step.outputs.artifact-id }}"
113+
114+
report:
115+
name: Report Job Status
116+
needs: [changes, formatting, test, docs]
117+
if: always()
118+
runs-on: ubuntu-24.04
119+
steps:
120+
- name: Report Job Success
121+
run: echo "All Tests Succeeded"
122+
if: |
123+
needs.formatting.result == 'success' &&
124+
(needs.test.result == 'skipped' || needs.test.result == 'success') &&
125+
(needs.docs.result == 'skipped' || needs.docs.result == 'success')
126+
- name: Report Job Failure
127+
run: |
128+
echo "Not All Tests Succeeded..."
129+
exit 1
130+
if: |
131+
needs.formatting.result != 'success' ||
132+
(needs.test.result != 'skipped' && needs.test.result == 'failure') ||
133+
(needs.docs.result != 'skipped' && needs.docs.result == 'failure')

0 commit comments

Comments
 (0)