Closed tomkyle closed 7 years ago
Currently, the parameter strings for command calls are constructed like this:
args="" if [ option ]; then args+=" --option" fi
This is antipattern, as deryni points out in a discussion about shellcheck being “too overzealous in regards to unquoted variables”
The better way to go is storing the options in a parameter, like described here: 5. I'm constructing a command based on information that is only known at run time
Closed as of 1.1.5
tomkyle
commented
7 years ago
Currently, the parameter strings for command calls are constructed like this:
This is antipattern, as deryni points out in a discussion about shellcheck being “too overzealous in regards to unquoted variables”
The better way to go is storing the options in a parameter, like described here: 5. I'm constructing a command based on information that is only known at run time