Skip to content

Commit

Permalink
Change Shellcheck config
Browse files Browse the repository at this point in the history
- Remove SC1091 "Not following source" as all sources should have a `# shellcheck source=...` directive.
- Add `disable=SC2230`  to allow non-standard `which` for older shellcheck versions (it is already disabled in newer versions).
- Add comments for optional checks we might want to enable in the near future.
  • Loading branch information
Potherca committed Apr 6, 2022
1 parent d149644 commit 1f27333
Showing 1 changed file with 22 additions and 2 deletions.
24 changes: 22 additions & 2 deletions .shellcheckrc
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
# ============================================================================ #
# Disable specific checks
# ---------------------------------------------------------------------------- #
# shellcheck disable=SC2034

## Allow: To read lines rather than words, pipe/redirect to a 'while read' loop
Expand All @@ -12,5 +15,22 @@ disable=SC2207
## Allow: Instead of 'let expr', prefer (( expr ))
disable=SC2219

## Allow: Not following source
disable=SC1091
#Allow: non-standard `which` in shellcheck < 0.7.1
disable=SC2230
# ============================================================================ #

# ============================================================================ #
# Enable optional checks (since 0.7.0).
# ---------------------------------------------------------------------------- #
## Suggest adding a default case in `case` statements
#enable=add-default-case

## Suggest explicitly using -n in `[ $var ]`
#enable=avoid-nullary-conditions

## Suggest quoting variables without metacharacters
#enable=quote-safe-variables

## Suggest putting braces around all variable references
#enable=require-variable-braces
# ============================================================================ #

0 comments on commit 1f27333

Please sign in to comment.