paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-10-25 00:53:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

tools: Remove -f short option for test-all.

Browse Source 
./tools/test-all had a short option -f which was used for running it
by skipping provision check. This short option paired well with its
long version --force, but in an effort to provide more clarity over
the use of force option it is renamed to --skip-provision-check.

Following this rename -f option for test-all does not pair well with
its long part which is now --skip-provision-check. Also, this option
is rarely used. So the short form -f is removed in this commit. We
now provide only its longer version --skip-provision-check.

This is a part of commits done to address issue #17455.
This commit is contained in:
m-e-l-u-h-a-n
2021-03-03 10:48:49 +05:30
committed by Tim Abbott
parent 63c12d2e59
commit ec8244b113
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tools/test-all
View File

@@ -4,13 +4,13 @@ set -e
cd "$(dirname "$0")"/.. cd "$(dirname "$0")"/..
# read the options # read the options
TEMP=$(getopt -o f --long force -- "$@") TEMP=$(getopt -o "" --long skip-provision-check -- "$@")
eval set -- "$TEMP" eval set -- "$TEMP"
# extract options. # extract options.
while true; do while true; do
case "$1" in case "$1" in
-f | --skip-provision-check) --skip-provision-check)
FORCEARG="--skip-provision-check" FORCEARG="--skip-provision-check"
shift shift
;; ;;