diff options
author | Ed Santiago <santiago@redhat.com> | 2021-01-07 13:57:35 -0700 |
---|---|---|
committer | Ed Santiago <santiago@redhat.com> | 2021-01-19 10:45:58 -0700 |
commit | 9f6bb3563d7a311a7e5628d18210c2c263af962d (patch) | |
tree | 9d208c129ff8ea39a0c2c0057b153896824ed949 /contrib/cirrus/runner.sh | |
parent | 8c6df5e93e6941a7f50da651678751f7dfec900e (diff) | |
download | podman-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-x | contrib/cirrus/runner.sh | 4 |
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 } |