Skip to content

Testing PR where to find CI#17

Open
TsuyoshiUshio wants to merge 13 commits intomasterfrom
somenew
Open

Testing PR where to find CI#17
TsuyoshiUshio wants to merge 13 commits intomasterfrom
somenew

Conversation

@TsuyoshiUshio
Copy link
Copy Markdown
Owner

@TsuyoshiUshio TsuyoshiUshio commented Jul 29, 2019

This is the sample of the Security Bot Pull Request Validation

return "this is not on the coding rules.";
}

private static string ChangePassword = "PassWord917231";
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

CODE_SMELL

4 more lines of code need to be covered by tests to reach the minimum threshold of 65.0% lines coverage.
See details

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

/falsepositive

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

The issue marked as falsepositive. For more detail.

return "this is not on the coding rules.";
}

private static string ChangePassword = "PassWord917231";
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

CODE_SMELL

10 more comment lines need to be written to reach the minimum threshold of 25.0% comment density.
See details

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

/workitem

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

WorkItem 906 Created see workItem.

return "this is not on the coding rules.";
}

private static string ChangePassword = "PassWord917231";
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

CODE_SMELL

Add or update the header of this file.
See details

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

/falsepositive

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

The issue marked as falsepositive. For more detail.

return "this is not on the coding rules.";
}

private static string ChangePassword = "PassWord917231";
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

CODE_SMELL

Remove the unused private field 'ChangePassword'.
See details

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

/workitem

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

WorkItem 907 Created see workItem.

return "this is not on the coding rules.";
}

private static string ChangePassword = "PassWord917231";
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

VULNERABILITY

'password' detected in this expression, review this potentially hardcoded credential.
See details

return "this is not on the coding rules.";
}

private static string ChangePassword = "PassWord917231";
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

CODE_SMELL

The field 'HomeController.ChangePassword' is assigned but its value is never used
See details

return "this is not on the coding rules.";
}

private static string ChangePassword = "PassWord917231";
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

CODE_SMELL

Remove this method and declare a constant for this value.
See details

@TsuyoshiUshio
Copy link
Copy Markdown
Owner Author

CODE_SMELL

[VulnerableApplication] Mark this assembly with 'System.CLSCompliantAttribute'
See details

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.

1 participant