Skip to content

Commit e05542e

Browse files
committed
style: fix more lint issues
1 parent 615614c commit e05542e

File tree

1 file changed

+5
-11
lines changed

1 file changed

+5
-11
lines changed

bin/proxy

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -170,9 +170,8 @@ function determine_virtual_host() {
170170
fi
171171

172172
local HOST
173-
HOST=$(pick_one_virtual_host_from_file $DOCKER_FILE)
174173

175-
if [ $? -ne 0 ]; then
174+
if ! HOST=$(pick_one_virtual_host_from_file "$DOCKER_FILE"); then
176175
echo "$HOST"
177176
exit 1
178177
fi
@@ -181,7 +180,7 @@ function determine_virtual_host() {
181180
}
182181

183182
function determine_virtual_host_override() {
184-
if [ -z $1 ]; then
183+
if [ -z "$1" ]; then
185184
output_error "No local host provided."
186185
exit 1
187186
fi
@@ -192,9 +191,8 @@ function determine_virtual_host_override() {
192191
fi
193192

194193
local HOST
195-
HOST=$(pick_one_virtual_host_from_file $DOCKER_FILE_OVERRIDE $1)
196194

197-
if [ $? -ne 0 ]; then
195+
if ! HOST=$(pick_one_virtual_host_from_file "$DOCKER_FILE_OVERRIDE" "$1"); then
198196
echo "$HOST"
199197
exit 1
200198
fi
@@ -203,9 +201,7 @@ function determine_virtual_host_override() {
203201
}
204202

205203
if [ -z "$LOCAL_DOMAIN" ]; then
206-
LOCAL_DOMAIN=$(determine_virtual_host)
207-
208-
if [ $? -ne 0 ]; then
204+
if ! LOCAL_DOMAIN=$(determine_virtual_host); then
209205
echo "$LOCAL_DOMAIN"
210206
exit 1
211207
fi
@@ -216,9 +212,7 @@ if [ -z "$LOCAL_PORT" ]; then
216212
fi
217213

218214
if [ -z "$PROXY_DOMAIN" ]; then
219-
PROXY_DOMAIN=$(determine_virtual_host_override "$LOCAL_DOMAIN")
220-
221-
if [ $? -ne 0 ]; then
215+
if ! PROXY_DOMAIN=$(determine_virtual_host_override "$LOCAL_DOMAIN"); then
222216
echo "$PROXY_DOMAIN"
223217
exit 1
224218
fi

0 commit comments

Comments
 (0)