ktmud commented on a change in pull request #12625:
URL: https://github.com/apache/superset/pull/12625#discussion_r563252497



##########
File path: .github/workflows/cancel_duplicates.yml
##########
@@ -1,25 +1,33 @@
 name: Cancel Duplicates
 on:
-  workflow_run:
-    workflows: ["CI"]
-    types: ["requested"]
+  # Checks for duplicate jobs on every push (merge) and every 10 minutes
+  push:
+  schedule:
+    - cron: "*/10 * * * *"
 
 jobs:
-  cancel-duplicate-workflow-runs:
-    name: "Cancel duplicate workflow runs"
-    runs-on: ubuntu-latest
+  cancel-duplicate-runs:
+    name: Cancel duplicate workflow runs
+    runs-on: ubuntu-20.04
+    continue-on-error: true

Review comment:
       There was actually two steps after the error before I migrated this to 
directly referring your workflow (since I need to checkout the code). And I 
also wasn't sure whether ` >= 20` will work for workflow `if` conditions.
   
   Thanks for the suggestion. I'll try it out!




----------------------------------------------------------------
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org

Reply via email to