You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@inlong.apache.org by GitBox <gi...@apache.org> on 2022/06/14 05:32:37 UTC

[GitHub] [incubator-inlong] shink commented on a diff in pull request #4654: [INLONG-4653][CI] Add concurrency support on GitHub Actions

shink commented on code in PR #4654:
URL: https://github.com/apache/incubator-inlong/pull/4654#discussion_r896389773


##########
.github/workflows/ci_docker.yml:
##########
@@ -90,7 +97,10 @@ jobs:
           fi
 
       - name: Push Docker images to Docker Hub
-        if: ${{ success() && steps.match-branch.outputs.match == 'true' }}
+        if: |
+          success()
+          && steps.check-workflow-diff.outputs.changed_only == 'no'
+          && steps.match-branch.outputs.match == 'true'

Review Comment:
   If only the workflow file (`.github/workflows/ci_docker.yml`) is changed, there is no need to publish images.
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org