Code

Resolve shellcheck lint issues#100

Code

Merged

BenWestgate merged 22 commits intoBenWestgate:masterBenWestgate/Bails:masterfrom epiccurious:95-resolve-shellcheck-lint-issuesepiccurious/Bails:95-resolve-shellcheck-lint-issuesCopy head branch name to clipboardMay 9, 2024

+37-32

Commits

Commits on May 3, 2024

• fix: exclude double-quoted variables rule

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  744c515

  Copy full SHA for 744c515

• refactor: fix lint issues in persistent-setup

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  a4bc637

  Copy full SHA for a4bc637

• refactor: increase indentation of onlynet_onion

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  bb98c52

  Copy full SHA for bb98c52

• refactor: satisfy shellcheck SC2181

  Show description for 6ecee17
  

  epiccurious

  authored and

  bitcoin-tools

  committed
  6ecee17

  Copy full SHA for 6ecee17

• refactor: fix indentation of link_dotfiles subshell

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  e54a538

  Copy full SHA for e54a538

• refactor: avoid for loops over find output [SC2044]

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  a88bf09

  Copy full SHA for a88bf09

• refactor: comment out unused variable

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  329ca51

  Copy full SHA for 329ca51

• refactor: ensure variable never expands to /bin or /lib

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  7149dc9

  Copy full SHA for 7149dc9

• refactor: use -n instead of ! -z

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  3a9ecfc

  Copy full SHA for 3a9ecfc

• Ignore SC2046 in CI checks

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  62e294d

  Copy full SHA for 62e294d

• fix syntax in yaml

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  7ecbb14

  Copy full SHA for 7ecbb14

• refactor: read without -r will mangle backslashes

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  ac02c08

  Copy full SHA for ac02c08

• refactor: Declare and assign separately to avoid masking return values

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  e5bfc0c

  Copy full SHA for e5bfc0c

• refactor: read without -r will mangle backslashes

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  1caf92f

  Copy full SHA for 1caf92f

• refactor: Don't use variables in the printf format string

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  f82a723

  Copy full SHA for f82a723

• refactor: $ is unnecessary on arithmetic variables

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  378e610

  Copy full SHA for 378e610

• Remove the repeated ', providing ... privacy.'

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  ab5cab0

  Copy full SHA for ab5cab0

• Clarify that Bitcoin Core connects, not Bails

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  6a4cacd

  Copy full SHA for 6a4cacd

• Make consistent with Lines 113 and 156

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  28d0d46

  Copy full SHA for 28d0d46

• Exclude SC2012 (Use 'cd ... || exit') and SC2164 (find instead of ls)

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  214f174

  Copy full SHA for 214f174

• wording change in comment

  

  BenWestgate

  authored and

  bitcoin-tools

  committed
  5cabbd6

  Copy full SHA for 5cabbd6

• lint fixes

  

  epiccurious

  authored and

  bitcoin-tools

  committed
  0391c05

  Copy full SHA for 0391c05