Showing 105 of 105 total issues
var parentId should be parentID Open
parentName, parentId := nameAndId(parent)
- Exclude checks
This backslash+linefeed is literal. Break outside single quotes if you just want to break the line. Open
go test \
- Read upRead up
- Exclude checks
This backslash+linefeed is literal. Break outside single quotes if you just want to break the line.
Problematic code:
var='This is long \
piece of text'
Correct code:
var='This is a long '\
'piece of text'
Rationale:
You have a single quoted string containing a backslash followed by a linefeed (newline). Unlike double quotes or unquoted strings, this has no special meaning. The string will contain a literal backslash and a linefeed.
If you wanted to break the line but not add a linefeed to the string, stop the single quote, break the line, and reopen it. This is demonstrated in the correct code.
If you wanted to break the line and also include the linefeed as a literal, you don't need a backslash:
var='This is a multi-line string
with an embedded linefeed'
Exceptions:
If you do want a string containing a literal backslash+linefeed combo, such as with sed
, you can [[ignore]] this warning.
Notice
Original content from the ShellCheck https://github.com/koalaman/shellcheck/wiki.
This backslash+linefeed is literal. Break outside single quotes if you just want to break the line. Open
-coverprofile=profiles/$(echo "PKG" | sed -e "s|./||" -e "s|/|_|g").out \
- Read upRead up
- Exclude checks
This backslash+linefeed is literal. Break outside single quotes if you just want to break the line.
Problematic code:
var='This is long \
piece of text'
Correct code:
var='This is a long '\
'piece of text'
Rationale:
You have a single quoted string containing a backslash followed by a linefeed (newline). Unlike double quotes or unquoted strings, this has no special meaning. The string will contain a literal backslash and a linefeed.
If you wanted to break the line but not add a linefeed to the string, stop the single quote, break the line, and reopen it. This is demonstrated in the correct code.
If you wanted to break the line and also include the linefeed as a literal, you don't need a backslash:
var='This is a multi-line string
with an embedded linefeed'
Exceptions:
If you do want a string containing a literal backslash+linefeed combo, such as with sed
, you can [[ignore]] this warning.
Notice
Original content from the ShellCheck https://github.com/koalaman/shellcheck/wiki.
var childId should be childID Open
childName, childId := nameAndId(child)
- Exclude checks
Line length Open
<img src="https://raw.githubusercontent.com/soumya92/barista/gh-pages/logo/128.png" height="128" width="128" alt="Logo" />
- Read upRead up
- Exclude checks
MD013 - Line length
Tags: line_length
Aliases: line-length Parameters: linelength, codeblocks, tables (number; default 80, boolean; default true)
This rule is triggered when there are lines that are longer than the configured line length (default: 80 characters). To fix this, split the line up into multiple lines.
This rule has an exception where there is no whitespace beyond the configured line length. This allows you to still include items such as long URLs without being forced to break them in the middle.
You also have the option to exclude this rule for code blocks and tables. To
do this, set the code_blocks
and/or tables
parameters to false.
Code blocks are included in this rule by default since it is often a requirement for document readability, and tentatively compatible with code rules. Still, some languages do not lend themselves to short lines.
Your code does not pass gofmt in 1 place. Go fmt your code! Open
// Copyright 2018 Google Inc.
- Exclude checks
Inline HTML Open
<img src="https://raw.githubusercontent.com/soumya92/barista/gh-pages/logo/128.png" height="128" width="128" alt="Logo" />
- Read upRead up
- Exclude checks
MD033 - Inline HTML
Tags: html
Aliases: no-inline-html
This rule is triggered whenever raw HTML is used in a markdown document:
Inline HTML header
To fix this, use 'pure' markdown instead of including raw HTML:
# Markdown header
Rationale: Raw HTML is allowed in markdown, but this rule is included for those who want their documents to only include "pure" markdown, or for those who are rendering markdown documents in something other than HTML.
Your code does not pass gofmt in 1 place. Go fmt your code! Open
// Copyright 2017 Google Inc.
- Exclude checks
var childId should be childID Open
oldName, childId := nameAndId(child)
- Exclude checks
This backslash+linefeed is literal. Break outside single quotes if you just want to break the line. Open
"PKG" \
- Read upRead up
- Exclude checks
This backslash+linefeed is literal. Break outside single quotes if you just want to break the line.
Problematic code:
var='This is long \
piece of text'
Correct code:
var='This is a long '\
'piece of text'
Rationale:
You have a single quoted string containing a backslash followed by a linefeed (newline). Unlike double quotes or unquoted strings, this has no special meaning. The string will contain a literal backslash and a linefeed.
If you wanted to break the line but not add a linefeed to the string, stop the single quote, break the line, and reopen it. This is demonstrated in the correct code.
If you wanted to break the line and also include the linefeed as a literal, you don't need a backslash:
var='This is a multi-line string
with an embedded linefeed'
Exceptions:
If you do want a string containing a literal backslash+linefeed combo, such as with sed
, you can [[ignore]] this warning.
Notice
Original content from the ShellCheck https://github.com/koalaman/shellcheck/wiki.
Your code does not pass gofmt in 1 place. Go fmt your code! Open
// Copyright 2017 Google Inc.
- Exclude checks
This backslash+linefeed is literal. Break outside single quotes if you just want to break the line. Open
-covermode=atomic \
- Read upRead up
- Exclude checks
This backslash+linefeed is literal. Break outside single quotes if you just want to break the line.
Problematic code:
var='This is long \
piece of text'
Correct code:
var='This is a long '\
'piece of text'
Rationale:
You have a single quoted string containing a backslash followed by a linefeed (newline). Unlike double quotes or unquoted strings, this has no special meaning. The string will contain a literal backslash and a linefeed.
If you wanted to break the line but not add a linefeed to the string, stop the single quote, break the line, and reopen it. This is demonstrated in the correct code.
If you wanted to break the line and also include the linefeed as a literal, you don't need a backslash:
var='This is a multi-line string
with an embedded linefeed'
Exceptions:
If you do want a string containing a literal backslash+linefeed combo, such as with sed
, you can [[ignore]] this warning.
Notice
Original content from the ShellCheck https://github.com/koalaman/shellcheck/wiki.
Your code does not pass gofmt in 1 place. Go fmt your code! Open
// Copyright 2020 Google Inc.
- Exclude checks
var rootId should be rootID Open
var rootId = identify(Root)
- Exclude checks
Expressions don't expand in single quotes, use double quotes for that. Open
'for try in `seq 1 3`; do
- Read upRead up
- Exclude checks
Expressions don't expand in single quotes, use double quotes for that.
Problematic code:
name=World
echo 'Hello $name'
Correct code:
name=World
echo "Hello $name"
Rationale:
Single quotes prevent expansion of everything, including variables and command substitution.
If you want to use the values of variables and such, use double quotes instead.
Note that if you have other items that needs single quoting, you can use both in a single word:
echo '$1 USD is '"$rate GBP"
Exceptions
If you want $stuff
to be a literal dollar sign followed by the characters "stuff", you can [[ignore]] this message.
ShellCheck tries to be smart about it, and won't warn when this is used with awk, perl and similar, but there are some inherent ambiguities like 'I have $1 in my wallet'
, which could be "one dollar" or "whatever's in the first parameter".
In the particular case of sed
, ShellCheck uses additional heuristics to try to separate cases like 's/$foo/bar/'
(failing to replace the variable $foo
) with from the false positives like '$d'
(delete last line). If you're still triggering these, consider being more generous with your spaces: use $ { s/foo/bar; }
instead of ${s/foo/bar/;}
Notice
Original content from the ShellCheck https://github.com/koalaman/shellcheck/wiki.
var thingId should be thingID Open
thingName, thingId := nameAndId(thing)
- Exclude checks
First header should be a top level header Open
## Contributor License Agreement
- Read upRead up
- Exclude checks
MD002 - First header should be a top level header
Tags: headers
Aliases: first-header-h1
Parameters: level (number; default 1)
This rule is triggered when the first header in the document isn't a h1 header:
## This isn't a H1 header
### Another header
The first header in the document should be a h1 header:
# Start with a H1 header
## Then use a H2 for subsections
Can't follow non-constant source. Use a directive to specify location. Open
[ -e "$CONFIG_DIR/barista/keys" ] && . "$CONFIG_DIR/barista/keys"
- Read upRead up
- Exclude checks
Can't follow non-constant source. Use a directive to specify location.
Problematic code:
. "$(find_install_dir)/lib.sh"
Correct code:
# shellcheck source=src/lib.sh
. "$(find_install_dir)/lib.sh"
Rationale:
ShellCheck is not able to include sourced files from paths that are determined at runtime. The file will not be read, potentially resulting in warnings about unassigned variables and similar.
Use a [[Directive]] to point shellcheck to a fixed location it can read instead.
Exceptions:
If you don't care that ShellCheck is unable to account for the file, specify # shellcheck source=/dev/null
.
Notice
Original content from the ShellCheck https://github.com/koalaman/shellcheck/wiki.
Your code does not pass gofmt in 1 place. Go fmt your code! Open
// Copyright 2017 Google Inc.
- Exclude checks
TODO found Open
// TODO: Argument parsing here.
- Exclude checks