This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch v1-10-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 8e582911f9576fad1827d613a7980700c3c0080e
Author: Jarek Potiuk <jarek.pot...@polidea.com>
AuthorDate: Mon Nov 2 00:01:20 2020 +0100
Turns failure of PR label when approved action into warning (#12017)
Sometimes (quite often really) when PR gets approved, the PR
gets merged rather quickly, without waiting for result of this
action. Or a new PR gets pushed quickly. In those cases PR will
not be found. But this is usually not a problem then and rather
than failing, we should simply print a warning and exit.
(cherry picked from commit a7a7cf22172ca92edfb62c373993ed974a298496)
---
.github/workflows/label_when_reviewed_workflow_run.yml | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/.github/workflows/label_when_reviewed_workflow_run.yml
b/.github/workflows/label_when_reviewed_workflow_run.yml
index 0fcab12..56bc835 100644
--- a/.github/workflows/label_when_reviewed_workflow_run.yml
+++ b/.github/workflows/label_when_reviewed_workflow_run.yml
@@ -72,7 +72,7 @@ jobs:
./scripts/ci/selective_ci_checks.sh
fi
- name: "Label when approved by committers for PRs that require full
tests"
- uses:
TobKed/label-when-approved-action@7872312da76508d29f98d4fa68843ea91754cc59 #
v1.2
+ uses:
TobKed/label-when-approved-action@4c5190fec5661e98d83f50bbd4ef9ebb48bd1194 #
v1.3
id: label-full-test-prs-when-approved-by-commiters
if: >
steps.selective-checks.outputs.run-tests == 'true' &&
@@ -101,7 +101,7 @@ jobs:
[the run](https://github.com/${{ github.repository
}}/actions/runs/${{ github.run_id }})
"}
- name: "Label when approved by committers for PRs that do not require
full tests"
- uses:
TobKed/label-when-approved-action@7872312da76508d29f98d4fa68843ea91754cc59 #
v1.2
+ uses:
TobKed/label-when-approved-action@4c5190fec5661e98d83f50bbd4ef9ebb48bd1194 #
v1.3
id: label-simple-test-prs-when-approved-by-commiters
if: >
steps.selective-checks.outputs.run-tests == 'true' &&
@@ -116,7 +116,7 @@ jobs:
Airflow. The committers might merge it or can add a label 'full
tests needed' and re-run it
to run all tests if they see it is needed!
- name: "Label when approved by committers for PRs that do not require
tests at all"
- uses:
TobKed/label-when-approved-action@7872312da76508d29f98d4fa68843ea91754cc59 #
v1.2
+ uses:
TobKed/label-when-approved-action@4c5190fec5661e98d83f50bbd4ef9ebb48bd1194 #
v1.3
id: label-no-test-prs-when-approved-by-commiters
if: steps.selective-checks.outputs.run-tests != 'true'
with:
