Skip to content

Conversation

@cyjseagull
Copy link
Member

No description provided.

@ghost
Copy link

ghost commented Jul 9, 2020

Warnings are found on analyzing the commit be57edb.

5 warnings:

We recommend to address them as possible, for example, update outdated dependencies, fix the tool's configuration, configure sider.yml, turn off unused tools, and so on.

If you are struggling with these errors or warnings, feel free to ask us via chat. 💬

check_commit_message()
{
if [ "${TRAVIS_PULL_REQUEST}" != "false" ]; then
local skip=$(curl -s https://api.github.com/repos/FISCO-BCOS/java-sdk/pulls/${TRAVIS_PULL_REQUEST} | grep "title\"" | grep "${skip_check_words}")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[ShellCheck] Declare and assign separately to avoid masking return values. (view)

Rule Severity Code
SC2155 warning 2155

References:

You can close this issue if no need to fix it. Learn more.

check_commit_message()
{
if [ "${TRAVIS_PULL_REQUEST}" != "false" ]; then
local skip=$(curl -s https://api.github.com/repos/FISCO-BCOS/java-sdk/pulls/${TRAVIS_PULL_REQUEST} | grep "title\"" | grep "${skip_check_words}")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[ShellCheck] Double quote to prevent globbing and word splitting. (view)

Rule Severity Code
SC2086 info 2086

References:

You can close this issue if no need to fix it. Learn more.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 9, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants