diff options
author | Chris Evich <cevich@redhat.com> | 2022-09-23 12:12:25 -0400 |
---|---|---|
committer | Chris Evich <cevich@redhat.com> | 2022-09-26 11:57:20 -0400 |
commit | 527fc409e59430aecfc02dc2bde2fea9785d6cea (patch) | |
tree | 477e0e77601550a1e7bde98cfb8749f679b00447 /contrib/cirrus/runner.sh | |
parent | cf6136ffa5ff12c1662cd7f41fa58d48b0616bb7 (diff) | |
download | podman-527fc409e59430aecfc02dc2bde2fea9785d6cea.tar.gz podman-527fc409e59430aecfc02dc2bde2fea9785d6cea.tar.bz2 podman-527fc409e59430aecfc02dc2bde2fea9785d6cea.zip |
Cirrus: Add golang code consistency check script
Depends on #15893
Fixes: #15913
Signed-off-by: Chris Evich <cevich@redhat.com>
Diffstat (limited to 'contrib/cirrus/runner.sh')
-rwxr-xr-x | contrib/cirrus/runner.sh | 20 |
1 files changed, 1 insertions, 19 deletions
diff --git a/contrib/cirrus/runner.sh b/contrib/cirrus/runner.sh index 5b1bc8d5c..c44251e2f 100755 --- a/contrib/cirrus/runner.sh +++ b/contrib/cirrus/runner.sh @@ -233,25 +233,7 @@ function _run_consistency() { SUGGESTION="run 'make generate-bindings' and commit all changes" ./hack/tree_status.sh make completions SUGGESTION="run 'make completions' and commit all changes" ./hack/tree_status.sh - - if [[ -z "$CIRRUS_TAG" ]] && \ - req_env_vars CIRRUS_CHANGE_IN_REPO CIRRUS_PR DEST_BRANCH - then - local base diffs regex i - # Prevent this check from detecting itself - i=i - msg "#####" - msg "Verifying no change adds new calls to ${i}o/${i}outil." - base=$(git merge-base $DEST_BRANCH $CIRRUS_CHANGE_IN_REPO) - diffs=$(git diff $base $CIRRUS_CHANGE_IN_REPO -- '*.go' ':^vendor/') - regex=$(echo -e "^(\\+.+${i}o/${i}outil)|(\\+.+${i}outil\\..+)") - if egrep -q "$regex"<<<"$diffs"; then - die "Found attempted use of deprecated ${i}outils: -$(egrep -B 5 -A 5 "$regex"<<<"$diffs")" - fi - else - msg "Skipping check for ${i}o/${i}outil addition." - fi + $SCRIPT_BASE/check_go_changes.sh } function _run_build() { |