6 SC2160

Joachim Ansorg edited this page 2021-11-12 19:41:32 +01:00

Table of Contents

• Instead of [ true ], just use true.
• Problematic code:
• Correct code:
• Rationale:
• Exceptions:

Instead of [ true ], just use true.

Problematic code:

if [ true ]
then
  echo "always triggers"
fi

Correct code:

if true
then
  echo "always triggers"
fi

Rationale:

This is a stylistic suggestion to use true instead of [ true ].

[ true ] seems to suggest that the value "true" is somehow relevant to the statement. This is not the case, it doesn't matter. You can replace it with [ false ] or [ wombat ], and it will still always be true:

String	In brackets	Outside brackets
true	true	true
false	true	false
wombat	true	unknown command

It's therefore better to use it without brackets, so that the "true" actually matters.

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.