simonmichael/hledger
1
1
Fork 0
mirror of https://github.com/simonmichael/hledger.git synced 2025-01-06 02:23:46 +03:00
Code Issues Projects Releases Wiki Activity

Do not use set -e in helper shell scripts

Browse Source 
It is not a substitute for proper error checking, it can easily cause
more trouble than good and it would be a burden for contributors and a
source for potential misbehavior. See this take on the topic for
example: http://mywiki.wooledge.org/BashFAQ/105
This commit is contained in:
Vladimir Zhelezov 2020-12-19 08:30:32 +01:00
parent dea35043bd
commit ac143aff6b
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
shell-completion/parse-commands.sh
View File

@ -1,7 +1,7 @@
#!/usr/bin/env bash #!/usr/bin/env bash
# Parse hledger's help and output all commands and command aliases in # Parse hledger's help and output all commands and command aliases in
# parenthesis. Do not output single letter command aliases, it's not useful. # parenthesis. Do not output single letter command aliases, it's not useful.
set -euo pipefail set -uo pipefail
declare commands_help declare commands_help
commands_help=$(hledger) commands_help=$(hledger)

2
shell-completion/parse-options.sh
View File

@ -2,7 +2,7 @@
# Parse hledger's help and output long options. Do not propose single letter # Parse hledger's help and output long options. Do not propose single letter
# completions. Options requiring an argument make that explicit by appending the # completions. Options requiring an argument make that explicit by appending the
# equal sign (=) # equal sign (=)
set -euo pipefail set -uo pipefail
declare subcommand=${1:-} declare subcommand=${1:-}
declare hledgerArgs=(--help) declare hledgerArgs=(--help)