ci: Remove noisy compliance webhook print (#53432)
Finn Evers
created
This is very noisy and does not provide much, the artifact link to the
report is sufficient for now.
Release Notes:
- N/A
Change summary
.github/workflows/compliance_check.yml | 7 +------
.github/workflows/release.yml | 14 ++------------
tooling/xtask/src/tasks/workflows/release.rs | 7 +------
3 files changed, 4 insertions(+), 24 deletions(-)
Detailed changes
@@ -52,18 +52,13 @@ jobs:
- name: send_compliance_slack_notification
if: always()
run: |
- REPORT_CONTENT=""
- if [ -f "target/compliance-report.md" ]; then
- REPORT_CONTENT=$(cat "target/compliance-report.md")
- fi
-
if [ "$COMPLIANCE_OUTCOME" == "success" ]; then
STATUS="✅ Scheduled compliance check passed for $COMPLIANCE_TAG"
else
STATUS="⚠️ Scheduled compliance check failed for $COMPLIANCE_TAG"
fi
- MESSAGE=$(printf "%s\n\nReport: %s\nPRs needing review: %s\n\n%s" "$STATUS" "$ARTIFACT_URL" "https://github.com/zed-industries/zed/pulls?q=is%3Apr+is%3Aclosed+label%3A%22PR+state%3Aneeds+review%22" "$REPORT_CONTENT")
+ MESSAGE=$(printf "%s\n\nReport: %s\nPRs needing review: %s" "$STATUS" "$ARTIFACT_URL" "https://github.com/zed-industries/zed/pulls?q=is%3Apr+is%3Aclosed+label%3A%22PR+state%3Aneeds+review%22")
curl -X POST -H 'Content-type: application/json' \
--data "$(jq -n --arg text "$MESSAGE" '{"text": $text}')" \
@@ -325,18 +325,13 @@ jobs:
- name: send_compliance_slack_notification
if: always()
run: |
- REPORT_CONTENT=""
- if [ -f "target/compliance-report.md" ]; then
- REPORT_CONTENT=$(cat "target/compliance-report.md")
- fi
-
if [ "$COMPLIANCE_OUTCOME" == "success" ]; then
STATUS="✅ Compliance check passed for $COMPLIANCE_TAG"
else
STATUS="❌ Compliance check failed for $COMPLIANCE_TAG"
fi
- MESSAGE=$(printf "%s\n\nReport: %s\nPRs needing review: %s\n\n%s" "$STATUS" "$ARTIFACT_URL" "https://github.com/zed-industries/zed/pulls?q=is%3Apr+is%3Aclosed+label%3A%22PR+state%3Aneeds+review%22" "$REPORT_CONTENT")
+ MESSAGE=$(printf "%s\n\nReport: %s\nPRs needing review: %s" "$STATUS" "$ARTIFACT_URL" "https://github.com/zed-industries/zed/pulls?q=is%3Apr+is%3Aclosed+label%3A%22PR+state%3Aneeds+review%22")
curl -X POST -H 'Content-type: application/json' \
--data "$(jq -n --arg text "$MESSAGE" '{"text": $text}')" \
@@ -695,18 +690,13 @@ jobs:
- name: send_compliance_slack_notification
if: always()
run: |
- REPORT_CONTENT=""
- if [ -f "target/compliance-report.md" ]; then
- REPORT_CONTENT=$(cat "target/compliance-report.md")
- fi
-
if [ "$COMPLIANCE_OUTCOME" == "success" ]; then
STATUS="✅ Compliance check passed for $COMPLIANCE_TAG"
else
STATUS="❌ Compliance check failed for $COMPLIANCE_TAG"
fi
- MESSAGE=$(printf "%s\n\nReport: %s\nPRs needing review: %s\n\n%s" "$STATUS" "$ARTIFACT_URL" "https://github.com/zed-industries/zed/pulls?q=is%3Apr+is%3Aclosed+label%3A%22PR+state%3Aneeds+review%22" "$REPORT_CONTENT")
+ MESSAGE=$(printf "%s\n\nReport: %s\nPRs needing review: %s" "$STATUS" "$ARTIFACT_URL" "https://github.com/zed-industries/zed/pulls?q=is%3Apr+is%3Aclosed+label%3A%22PR+state%3Aneeds+review%22")
curl -X POST -H 'Content-type: application/json' \
--data "$(jq -n --arg text "$MESSAGE" '{"text": $text}')" \
@@ -177,18 +177,13 @@ pub(crate) fn add_compliance_notification_steps(
};
let script = formatdoc! {r#"
- REPORT_CONTENT=""
- if [ -f "{COMPLIANCE_REPORT_FILE}" ]; then
- REPORT_CONTENT=$(cat "{COMPLIANCE_REPORT_FILE}")
- fi
-
if [ "$COMPLIANCE_OUTCOME" == "success" ]; then
STATUS="{success_prefix} for $COMPLIANCE_TAG"
else
STATUS="{failure_prefix} for $COMPLIANCE_TAG"
fi
- MESSAGE=$(printf "%s\n\nReport: %s\nPRs needing review: %s\n\n%s" "$STATUS" "$ARTIFACT_URL" "{NEEDS_REVIEW_PULLS_URL}" "$REPORT_CONTENT")
+ MESSAGE=$(printf "%s\n\nReport: %s\nPRs needing review: %s" "$STATUS" "$ARTIFACT_URL" "{NEEDS_REVIEW_PULLS_URL}")
curl -X POST -H 'Content-type: application/json' \
--data "$(jq -n --arg text "$MESSAGE" '{{"text": $text}}')" \