2 SC2201

koalaman edited this page 2016-12-31 14:00:14 -08:00

Table of Contents

• Brace expansion doesn't happen in [[ ]]. Use a loop.
• Problematic code:
• Correct code:
• Rationale:
• Exceptions:

Brace expansion doesn't happen in [[ ]]. Use a loop.

Problematic code:

[[ "$file" = index.{htm,html,php} ]] && echo "This is the main file"

Correct code:

for main in index.{htm,html,php}
do
  [[ "$file" = "$main" ]] && echo "This is the main file"
done

Rationale:

Brace expansions doesn't happen in [[ ]]. They will just be interpreted literally.

Instead, use a for loop to iterate over values, and apply your condition to each.

Exceptions:

None.

ShellCheck

• Installation

  • CentOS6
  • More Installation Guides

• Usage

  • Directive
  • Ignore

• Integrating and extending

  • Misc. contributions
  • Azure Pipelines
  • Buildkite
  • CircleCI
  • DevGuide
  • Integration
  • JUnit
  • Phabricator
  • TravisCI
  • GitLab CI
  • Drone
  • GitHub Actions

• Severity

• Template

Each individual ShellCheck warning has its own wiki page like SC1000. Use GitHub Wiki's "Pages" feature above to find a specific one, or see Checks.