mik-laj commented on a change in pull request #31104:
URL: https://github.com/apache/spark/pull/31104#discussion_r554447297
##########
File path: .github/workflows/build_and_test.yml
##########
@@ -89,6 +89,14 @@ jobs:
# In order to fetch changed files
with:
fetch-depth: 0
+ - uses: actions/checkout@v2
+ name: checkout n1hility/cancel-previous-runs
+ with:
+ repository: n1hility/cancel-previous-runs
+ path: ./build/.actions/n1hility/cancel-previous-runs
+ ref: 953c92201f368370112ea2754545cb4468d89f12 # v2
+ fetch-depth: 1
+ - uses: ./build/.actions/n1hility/cancel-previous-runs
Review comment:
I agree. I checked carefully and it looks like this action must be run
as workflow_run.
> If you use forks, you should create a separate "Cancelling" workflow_run
triggered workflow. The workflow_run should be responsible for all canceling
actions. The examples below show the possible ways the action can be utilized.
https://github.com/potiuk/cancel-workflow-runs#usage
I am rewriting my PR now to use workflow_run. I will also ask @potiuk for a
review when I'm finished.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
