commit 297c335017359bcc5083683de87753fb197bee36 Author: aaron Date: Thu Oct 27 17:09:13 2022 +0200 add best practices diff --git a/README.md b/README.md new file mode 100644 index 0000000..088699c --- /dev/null +++ b/README.md @@ -0,0 +1,33 @@ +# Shell Script Best Practices + +- Use `bash`. Using `zsh` or `fish` or any other, will make it hard for others to understand / collaborate. Among all shells, bash strikes a good balance between portability and DX. +- Just make the first line be `#!/usr/bin/env bash`, even if you don’t give executable permission to the script file. +- Use the .sh (or .bash) extension for your file. It may be fancy to not have an extension for your script, but unless your case explicitly depends on it, you’re probably just trying to do clever stuff. Clever stuff are hard to understand. +- Use `set -o errexit` at the start of your script. + - So that when a command fails, bash exits instead of continuing with the rest of the script. +- Prefer to use `set -o nounset`. You may have a good excuse to not do this, but, my opinion, it’s best to always set it. + - This will make the script fail, when accessing an unset variable. Saves from horrible unintended consequences, with typos in variable names. + - When you want to access a variable that may or may not have been set, use `"${VARNAME-}"` instead of `"$VARNAME"`, and you’re good. +- Use set `-o pipefail`. Again, you may have good reasons to not do this, but I’d recommend to always set it. + - This will ensure that a pipeline command is treated as failed, even if one command in the pipeline fails. +- Use `set -o xtrace`, with a check on `$TRACE` env variable. + - For copy-paste: `if [[ "${TRACE-0}" == "1" ]]; then set -o xtrace; fi`. + - This helps in debugging your scripts, a lot. Like, really lot. + - People can now enable debug mode, by running your script as `TRACE=1 ./script.sh` instead of `./script.sh.` +- Use `[[ ]]` for conditions in if `/` while statements, instead of `[ ]` or test. + - `[[ ]]` is a bash builtin, and is more powerful than `[ ]` or test. +- Always quote variable accesses with double-quotes. + - One place where it’s okay not to is on the left-hand-side of an `[[ ]]` condition. But even there I’d recommend quoting. + - When you need the unquoted behaviour, using bash arrays will likely serve you much better. +- Use local variables in functions. +- Accept multiple ways that users can ask for help and respond in kind. + - Check if the first arg is -h or --help or help or just h or even -help, and in all these cases, print help text and exit. + - Please. For the sake of your future-self. +- When printing error messages, please redirect to stderr. + - Use echo 'Something unexpected happened' `>&2` for this. +- Use long options, where possible (like `--silent` instead of `-s`). These serve to document your commands explicitly. + - Note though, that commands shipped on some systems like macOS don’t always have long options. +- If appropriate, change to the script’s directory close to the start of the script. + - And it’s usually always appropriate. + - Use cd `"$(dirname "$0")"`, which works in most cases. +- Use shellcheck. Heed its warnings. diff --git a/script.sh b/script.sh new file mode 100755 index 0000000..1fec152 --- /dev/null +++ b/script.sh @@ -0,0 +1,26 @@ +#!/usr/bin/env bash + +set -o errexit +set -o nounset +set -o pipefail + +if [[ "${TRACE-0}" == "1" ]]; then + set -o xtrace +fi + +if [[ "${1-}" =~ ^-*h(elp)?$ ]]; then + echo 'Usage: ./script.sh arg-one arg-two + +This is an awesome bash script to make your life better. + +' + exit +fi + +cd "$(dirname "$0")" + +main() { + echo do awesome stuff +} + +main "[email protected]"