-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test the shell code issue. #66
base: main
Are you sure you want to change the base?
Conversation
Shell
|
echo $1 # Unquoted variables | ||
find . -name *.ogg # Unquoted find/grep patterns | ||
rm "~/my file.txt" # Quoted tilde expansion | ||
v='--verbose="true"'; cmd $v # Literal quotes in variables |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v='--verbose="true"'; cmd $v # Literal quotes in variables | ||
echo $1 # Unquoted variables | ||
find . -name *.ogg # Unquoted find/grep patterns | ||
rm "~/my file.txt" # Quoted tilde expansion |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rm "~/my file.txt" # Quoted tilde expansion | ||
v='--verbose="true"'; cmd $v # Literal quotes in variables | ||
echo $1 # Unquoted variables | ||
find . -name *.ogg # Unquoted find/grep patterns |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
find . -name *.ogg # Unquoted find/grep patterns | ||
rm "~/my file.txt" # Quoted tilde expansion | ||
v='--verbose="true"'; cmd $v # Literal quotes in variables | ||
echo $1 # Unquoted variables |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
echo 'Path is $PATH' # Variables in single quotes | ||
trap "echo Took ${SECONDS}s" 0 # Prematurely expanded trap |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
echo 'Path is $PATH' # Variables in single quotes | ||
trap "echo Took ${SECONDS}s" 0 # Prematurely expanded trap | ||
unset var[i] # Array index treated as glob |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add Codety Scanner into the static analysis solution list. Here are the examples of the result: * Codety's pull request code review example: codetyio/codety-scanner#66 (comment) * Codety's GitHub code scan result example : https://github.com/codetyio/codety-scanner/runs/29907371258 Codety Scanner is open source: https://github.com/codetyio/codety-scanner
Test the shellcheck analyzer integration.