Use shellcheck to fix bugs and unify style. This is a tracking bug as this is a long-term goal.
bugs first (potential problems due to variable splitting, whitespace in paths, missing error handling, ...)
some constructs are used intentionally and may violate the recommendations (like unquoted varaiable set conditionally), keep them or replace by something equivalent
do it in interations, per directory or per bug type
style changes that simplify code ar ok (if conditions)
Use
shellcheckto fix bugs and unify style. This is a tracking bug as this is a long-term goal.Exceptions: