fix ci results comment

elastic · Mar 28, 2024 · be82d40 · be82d40
1 parent 2451b25
commit be82d40
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -1,7 +1,7 @@
 name: Tests
 
 on:
-  pull_request_target:
+  pull_request:
     branches:
       - main
       - "[0-9]+.[0-9]+"
@@ -320,6 +320,7 @@ jobs:
         run: |
           ls -lahR || true
           rm -f tests/allure/results/cover.out || true
+          echo "${{ github.event_name }}"
 
       - name: Publish allure report
         if: ${{ success() || failure() }}
@@ -334,7 +335,7 @@ jobs:
           resultsGlob: "tests/allure/results"
           updatePr: actions
           collapseSummary: false
-          summary: suites
+          summary: packages
           summaryTableType: markdown
           copyLatest: true
           bucket: csp-allure-reports
@@ -343,7 +344,7 @@ jobs:
           debug: false
 
       - name: Allure Summary
-        if: ${{ success() && github.event_name == 'pull_request' }}
+        if: ${{ success() && github.event_name != 'push' }}
         env:
           GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -354,8 +355,6 @@ jobs:
 
           ./.ci/scripts/allure-report-summary.sh "$allure_result" "$allure_url"
           allure_summary=$(./.ci/scripts/allure-report-summary.sh "$allure_result" "$allure_url")
-          # posting result into job summary
-          echo "$allure_summary" >> $GITHUB_STEP_SUMMARY
 
           # saving result into env variable (with multiline handling)
           echo "ALLURE_SUMMARY<<EOF" >> $GITHUB_ENV
@@ -364,7 +363,7 @@ jobs:
 
       - name: Comment test success result
         uses: marocchino/sticky-pull-request-comment@v2
-        if: ${{ success() && github.event_name == 'pull_request' }}
+        if: ${{ success() && github.event_name != 'push' }}
         with:
           header: CI Test Results
           number: ${{ github.event.number }}