From 23a2b55eaa4e9b9a1f967c600480a8b2fc557867 Mon Sep 17 00:00:00 2001 From: toonn Date: Sat, 24 Jul 2021 18:22:14 +0200 Subject: Actions: Add on pull_request again MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit My testing was flawed. The status checks seemed to work fine because the builds were triggered on push in the same repo. But this doesn't work across repos. So GH Actions having run to completion on someone's fork doesn't give us the status checks we need. Therefore we want to run our Actions on PRs too, that way they're run from the ranger repo as well and the status checks should actually be informative. One possible improvement is that we could duplicate the workflows we want to run on PRs, that way we can drop all the things that don't really matter for status checks from the version matrices. Otoh that means we have pretty much duplicated workflows we need to keep in sync. Thank you for your lack of abstractions, GH Actions 🙄 --- .github/workflows/pylint.yml | 4 ++++ .github/workflows/python.yml | 4 ++++ .github/workflows/shellcheck.yml | 4 ++++ 3 files changed, 12 insertions(+) diff --git a/.github/workflows/pylint.yml b/.github/workflows/pylint.yml index 349c9af4..eac9aa5c 100644 --- a/.github/workflows/pylint.yml +++ b/.github/workflows/pylint.yml @@ -5,6 +5,10 @@ on: paths: - '.github/workflows/pylint.yml' - '**.py' + pull_request: + paths: + - '.github/workflows/pylint.yml' + - '**.py' jobs: test_pylint: diff --git a/.github/workflows/python.yml b/.github/workflows/python.yml index 82672bfb..b96d6812 100644 --- a/.github/workflows/python.yml +++ b/.github/workflows/python.yml @@ -5,6 +5,10 @@ on: paths: - '.github/workflows/python.yml' - '**.py' + pull_request: + paths: + - '.github/workflows/python.yml' + - '**.py' jobs: test_py: diff --git a/.github/workflows/shellcheck.yml b/.github/workflows/shellcheck.yml index cd718a2d..783ff0fa 100644 --- a/.github/workflows/shellcheck.yml +++ b/.github/workflows/shellcheck.yml @@ -5,6 +5,10 @@ on: paths: - '.github/workflows/shellcheck.yml' - 'ranger/data/scope.sh' + pull_request: + paths: + - '.github/workflows/shellcheck.yml' + - 'ranger/data/scope.sh' jobs: test_shellcheck: -- cgit 1.4.1-2-gfad0