summaryrefslogtreecommitdiff
path: root/contrib/cirrus/runner.sh
diff options
context:
space:
mode:
authorEd Santiago <santiago@redhat.com>2021-01-07 13:57:35 -0700
committerEd Santiago <santiago@redhat.com>2021-01-19 10:45:58 -0700
commit9f6bb3563d7a311a7e5628d18210c2c263af962d (patch)
tree9d208c129ff8ea39a0c2c0057b153896824ed949 /contrib/cirrus/runner.sh
parent8c6df5e93e6941a7f50da651678751f7dfec900e (diff)
downloadpodman-9f6bb3563d7a311a7e5628d18210c2c263af962d.tar.gz
podman-9f6bb3563d7a311a7e5628d18210c2c263af962d.tar.bz2
podman-9f6bb3563d7a311a7e5628d18210c2c263af962d.zip
CI: smoke test: insist on adding tests on PRs
On each PR (with a few exceptions), check the list of git-touched files, and abort if no tests are added. Include instructions on how to bypass the check if tests really aren't needed. Include a hardcoded exception list for PRs that only touch a well-known subset of "safe" files: docs, .cirrus.yml, vendor, version, hack, contrib, or *.md. This list is likely to need tuning over time. Add a test suite, but not one recognized by the new script (because it's a "*.t" file), so: [NO TESTS NEEDED] Signed-off-by: Ed Santiago <santiago@redhat.com>
Diffstat (limited to 'contrib/cirrus/runner.sh')
-rwxr-xr-xcontrib/cirrus/runner.sh4
1 files changed, 4 insertions, 0 deletions
diff --git a/contrib/cirrus/runner.sh b/contrib/cirrus/runner.sh
index b0060163e..e08bb5f19 100755
--- a/contrib/cirrus/runner.sh
+++ b/contrib/cirrus/runner.sh
@@ -31,7 +31,11 @@ function _run_smoke() {
# $CIRRUS_TAG is only non-empty when executing due to a tag-push
# shellcheck disable=SC2154
if [[ -z "$CIRRUS_TAG" ]]; then
+ # If PR consists of multiple commits, test that each compiles cleanly
make .gitvalidation
+
+ # PRs should include some way to test.
+ $SCRIPT_BASE/pr-should-include-tests
fi
}