SC2266

Jump to bottom Edit New page

Vidar Holen edited this page Nov 3, 2022 · 1 revision

Use && for logical AND. Single & will background and return true.

Problematic code:

if [ "$1" = "--verbose" ] | [ "$1" = "-v" ]
then
  verbose=1
fi

Correct code:

if [ "$1" = "--verbose" ] || [ "$1" = "-v" ]
then
  verbose=1
fi

Rationale:

ShellCheck found a test command followed by a |. This was undoubtedly intended as a logical OR (||).

Exceptions:

None

Related resources:

Help by adding links to BashFAQ, StackOverflow, man pages, POSIX, etc!

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.

Clone this wiki locally