ci: Reduce compliance notifications on success (#54669)
...What a beatiful syntax to have in a context where anything else would not be evaluated regardless. Release Notes: - N/A
This commit is contained in:
1 parent
bb47d90c76
commit
7d8e34ae14
3 files changed
+5
-5
No files matched your search
@@ -50,7 +50,7 @@ jobs:
|
||||
path: compliance-report-${{ github.ref_name }}.md
|
||||
if-no-files-found: error
|
||||
- name: send_compliance_slack_notification
|
||||
if: always()
|
||||
if: ${{ always() }}
|
||||
run: |
|
||||
if [ "$COMPLIANCE_OUTCOME" == "success" ]; then
|
||||
STATUS="✅ Scheduled compliance check passed for $COMPLIANCE_TAG"
|
||||
|
||||
@@ -322,7 +322,7 @@ jobs:
|
||||
path: compliance-report-${{ github.ref_name }}.md
|
||||
if-no-files-found: error
|
||||
- name: send_compliance_slack_notification
|
||||
if: always()
|
||||
if: ${{ always() }}
|
||||
run: |
|
||||
if [ "$COMPLIANCE_OUTCOME" == "success" ]; then
|
||||
STATUS="✅ Compliance check passed for $COMPLIANCE_TAG"
|
||||
@@ -696,7 +696,7 @@ jobs:
|
||||
if-no-files-found: error
|
||||
overwrite: true
|
||||
- name: send_compliance_slack_notification
|
||||
if: failure() || needs.compliance_check.outputs.outcome != 'success'
|
||||
if: ${{ failure() || needs.compliance_check.outputs.outcome != 'success' }}
|
||||
run: |
|
||||
if [ "$COMPLIANCE_OUTCOME" == "success" ]; then
|
||||
STATUS="✅ Compliance check passed for $COMPLIANCE_TAG"
|
||||
|
||||
Reference in new issue
Block a user