search

franciscolourenco / done

A fish-shell package to automatically receive notifications when long processes finish.
MIT License
767 stars 70 forks source link

Enclose test argument with double-quotes #59

Closed ammgws closed 4 years ago

ammgws commented 4 years ago

Found this when investigating the cause of https://github.com/fish-shell/fish-shell/issues/6350.

From fish docs:

If the variable is not set, set but with no value, or set to more than one value you must enclose it in double-quotes.

franciscolourenco commented 4 years ago

We already enclosed a few other variables in other test clauses, guess this one escaped. Thanks!