Skip to content
This repository was archived by the owner on Oct 1, 2020. It is now read-only.

Fixes bash syntax - #101

Open
asbarron wants to merge 1 commit into
spotify:masterfrom
asbarron:master
Open

Fixes bash syntax#101
asbarron wants to merge 1 commit into
spotify:masterfrom
asbarron:master

Conversation

@asbarron

@asbarron asbarron commented Oct 7, 2019

Copy link
Copy Markdown

Fix quoted variables and negated test -z resulting in a needless double-negative. You can just use the -n operator instead:

# Identical tests to verify that a value is assigned
[ ! -z foo ] # Not has no value
[ -n foo ]   # Has value

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant