Skip to content
This repository has been archived by the owner on Jul 8, 2020. It is now read-only.

example(test): Add some errors to the code #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

olivernybroe
Copy link
Contributor

No description provided.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Found 19 issues in the code.
PHP Insights has some concerns, please look into it.

Code Complexity Architecture Style
96.1% 91.3% 82.6% 95.1%

@@ -223,7 +223,8 @@ private static function getDescription(Results $results, string $reviewStatus, i
if ($reviewStatus === Review::COMMENT) {
return "{$prepend}PHP Insights has some concerns, please look into it.\n{$table}";
}

return "{$prepend}PHP Insights is not happy, please look into the comments, so we can be friends again.\n{$table}";
else {

Choose a reason for hiding this comment

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

[No else] Do not use "else/elseif". Prefer early return statement instead.


return "{$prepend}PHP Insights is not happy, please look into the comments, so we can be friends again.\n{$table}";
else {
return "{$prepend}PHP Insights is not happy, please look into the comments, so we can be friends again.\n{$table}";

Choose a reason for hiding this comment

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

[Line length] Line exceeds maximum limit of 120 characters; contains 127 characters

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant