nopolabs/yabot

View on GitHub
src/Plugin/MethodMatcher.php

Summary

Maintainability
A
50 mins
Test Coverage

Method __construct has 7 arguments (exceeds 4 allowed). Consider refactoring.
Open

        string $name,
        $isBot,
        array $channels,
        array $users,
        array $patterns,
Severity: Major
Found in src/Plugin/MethodMatcher.php - About 50 mins to fix

    Parameter $logger has undeclared type ?\Psr\Log\LoggerInterface
    Open

        public function __construct(
    Severity: Minor
    Found in src/Plugin/MethodMatcher.php by phan

    Possibly zero references to use statement for classlike/namespace Message (\Nopolabs\Yabot\Message\Message)
    Open

    use Nopolabs\Yabot\Message\Message;
    Severity: Minor
    Found in src/Plugin/MethodMatcher.php by phan

    The closing parenthesis of a multi-line function declaration must be on a new line
    Open

            LoggerInterface $logger = null)
    Severity: Minor
    Found in src/Plugin/MethodMatcher.php by phpcodesniffer

    There must be one blank line after the namespace declaration
    Open

    namespace Nopolabs\Yabot\Plugin;
    Severity: Minor
    Found in src/Plugin/MethodMatcher.php by phpcodesniffer

    The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line
    Open

        {
    Severity: Minor
    Found in src/Plugin/MethodMatcher.php by phpcodesniffer

    Expected 1 newline at end of file; 0 found
    Open

    }
    Severity: Minor
    Found in src/Plugin/MethodMatcher.php by phpcodesniffer

    There are no issues that match your filters.

    Category
    Status