[
https://issues.apache.org/jira/browse/BEAM-10404?focusedWorklogId=455297&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-455297
]
ASF GitHub Bot logged work on BEAM-10404:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 07/Jul/20 08:41
Start Date: 07/Jul/20 08:41
Worklog Time Spent: 10m
Work Description: TobKed commented on a change in pull request #12166:
URL: https://github.com/apache/beam/pull/12166#discussion_r450702973
##########
File path: .github/workflows/cancel.yml
##########
@@ -16,7 +16,7 @@
# under the License.
name: Cancel
-on: [push]
+on: [push, pull_request]
Review comment:
In relation to :
https://github.com/apache/beam/blob/master/.github/workflows/build_wheels.yml#L21
Push: when new commits are pushed to the specified branches. e.g. new merge
commit on master appear. So it is possible to run rests without PR.
Pull request: when PR to specified branch is created and then again when new
commits appear in the PR.
It is not critical since PR builds do not delete/upload anything to the GCS
(push builds do), but I think it would be convenient to decrease queue by
introducing such a small change.
----------------------------------------------------------------
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]
Issue Time Tracking
-------------------
Worklog Id: (was: 455297)
Time Spent: 40m (was: 0.5h)
> Cancel queued/running GitHub Action builds on second push to PR
> ---------------------------------------------------------------
>
> Key: BEAM-10404
> URL: https://issues.apache.org/jira/browse/BEAM-10404
> Project: Beam
> Issue Type: Sub-task
> Components: build-system
> Reporter: Tobiasz Kedzierski
> Assignee: Tobiasz Kedzierski
> Priority: P2
> Labels: CI, CI/CD
> Time Spent: 40m
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
