Merge pull request #45308 from fabianofranz/more_cmd_sanity_checks
Automatic merge from submit-queue (batch tested with PRs 46022, 46055, 45308, 46209, 43590) More cli sanity verifications Adds some more `kubectl` command sanity checks to improve consistency and avoid the need of code reviews for some of our CLI style and standards. **Release note**: ```release-note NONE ``` @kubernetes/sig-cli-pr-reviews
This commit is contained in:
@@ -32,9 +32,12 @@ clicheck=$(kube::util::find-binary "clicheck")
|
||||
|
||||
if ! output=`$clicheck 2>&1`
|
||||
then
|
||||
echo "FAILURE: CLI is not following one or more required conventions:"
|
||||
echo "$output"
|
||||
echo
|
||||
echo "FAILURE: CLI is not following one or more required conventions."
|
||||
exit 1
|
||||
else
|
||||
echo "$output"
|
||||
echo
|
||||
echo "SUCCESS: CLI is following all tested conventions."
|
||||
fi
|
||||
|
Reference in New Issue
Block a user