Issues (1)

src/Command/FixCommand.php (1 issue)

Severity
1
<?php
2
3
declare(strict_types=1);
4
5
namespace PK\CI\Command;
6
7
use PK\CI\Configuration\DirectoriesInterface;
8
use PK\CI\TestRunner\FixableRunnerInterface;
9
use PK\CI\TestRunner\RunnerInterface;
10
11
class FixCommand extends RunCommand
12
{
13
    /**
14
     * @param FixableRunnerInterface $runner
15
     *
16
     * {@inheritdoc}
17
     */
18
    protected function getArgv(RunnerInterface $runner, DirectoriesInterface $directories): array
19
    {
20
        return $runner->getArgv($this->locator, $directories, true);
0 ignored issues
show
The call to PK\CI\TestRunner\RunnerInterface::getArgv() has too many arguments starting with true. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

20
        return $runner->/** @scrutinizer ignore-call */ getArgv($this->locator, $directories, true);

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
21
    }
22
}
23