You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ro...@apache.org on 2022/04/20 11:09:11 UTC

[cloudstack] branch main updated: .github: fix coverage issue, don't kick pkging automatically

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/main by this push:
     new 40193a04ebc .github: fix coverage issue, don't kick pkging automatically
40193a04ebc is described below

commit 40193a04ebcfe2506279985f1208eb3855b18442
Author: Rohit Yadav <ro...@shapeblue.com>
AuthorDate: Wed Apr 20 16:37:32 2022 +0530

    .github: fix coverage issue, don't kick pkging automatically
    
    Signed-off-by: Rohit Yadav <ro...@shapeblue.com>
---
 .github/workflows/coverage-check.yml | 29 ++++++++---------------------
 1 file changed, 8 insertions(+), 21 deletions(-)

diff --git a/.github/workflows/coverage-check.yml b/.github/workflows/coverage-check.yml
index 5ff1dfcf06b..fc53ae14081 100644
--- a/.github/workflows/coverage-check.yml
+++ b/.github/workflows/coverage-check.yml
@@ -29,19 +29,6 @@ jobs:
           ref: "refs/pull/${{ github.event.number }}/merge"
           fetch-depth: 0
 
-      - name: Debug Github event
-        env: { CONTENT : "${{ toJson(github) }}" }
-        run: "echo $CONTENT"
-
-      - name: Changed Java Files
-        id: java-changes
-        uses: tj-actions/changed-files@a59f800cbb60ed483623848e31be67659a2940f8
-        with:
-          sha: "${{ github.pull_request.sha }}"
-          base_sha: "${{ github.sha }}"
-          files: |
-            **/*.java
-
       - name: Changed UI Files
         id: ui-changes
         uses: tj-actions/changed-files@a59f800cbb60ed483623848e31be67659a2940f8
@@ -61,15 +48,15 @@ jobs:
             Found UI changes, kicking a new UI QA build
             @blueorangutan ui
 
-      - name: Kick Pkg Build
-        uses: peter-evans/create-or-update-comment@c9fcb64660bc90ec1cc535646af190c992007c32
-        if: steps.java-changes.outputs.any_changed == 'true' && ! github.event.pull_request.draft
+      - name: Changed Java Files
+        id: java-changes
+        uses: tj-actions/changed-files@a59f800cbb60ed483623848e31be67659a2940f8
         with:
-          token: ${{ secrets.ACS_ROBOT_TOKEN }}
-          issue-number: ${{ github.event.pull_request.number }}
-          body: |
-            Found Java/XML changes, kicking packaging job
-            @blueorangutan package
+          sha: "${{ github.pull_request.sha }}"
+          base_sha: "${{ github.sha }}"
+          files: |
+            **/*.java
+            **/*.xml
 
       - name: Set up JDK11
         uses: actions/setup-java@v3