ci: add rector #39
neilimepull-request-ci.yml
on: pull_request
Matrix: Continuous Integration / checks
Annotations
12 errors and 1 warning
|
Continuous Integration / checks (8.3, true):
src/CssLint/CliArgs.php#L42
StrStartsWithRector / NullToStrictStringFuncCallArgRector
--- Original
+++ New
@@ -42,13 +42,13 @@
foreach ($aArguments as $sArgument) {
// --foo --bar=baz
- if (substr($sArgument, 0, 2) == '--') {
- $sEqualPosition = strpos($sArgument, '=');
+ if (str_starts_with((string) $sArgument, '--')) {
+ $sEqualPosition = strpos((string) $sArgument, '=');
// --bar=baz
if ($sEqualPosition !== false) {
- $sKey = substr($sArgument, 2, $sEqualPosition - 2);
- $sValue = substr($sArgument, $sEqualPosition + 1);
+ $sKey = substr((string) $sArgument, 2, $sEqualPosition - 2);
+ $sValue = substr((string) $sArgument, $sEqualPosition + 1);
$aParsedArguments[$sKey] = $sValue;
}
}
|
|
Continuous Integration / checks (8.3, true):
src/CssLint/Linter.php#L389
StrStartsWithRector
--- Original
+++ New
@@ -389,7 +389,7 @@
$sPropertyName = trim($this->getContextContent());
// Ignore CSS variables (names starting with --)
- if (substr($sPropertyName, 0, 2) === '--') {
+ if (str_starts_with($sPropertyName, '--')) {
$this->setContext(self::CONTEXT_PROPERTY_CONTENT);
return true;
}
|
|
Continuous Integration / checks (8.3, true):
src/CssLint/Cli.php#L7
Property CssLint\Cli::$SCRIPT_NAME has no type specified.
|
|
Continuous Integration / checks (8.3, true):
src/CssLint/Cli.php#L8
Property CssLint\Cli::$RETURN_CODE_ERROR has no type specified.
|
|
Continuous Integration / checks (8.3, true):
src/CssLint/Cli.php#L9
Property CssLint\Cli::$RETURN_CODE_SUCCESS has no type specified.
|
|
Continuous Integration / checks (8.3, true):
src/CssLint/Cli.php#L16
Method CssLint\Cli::run() has parameter $aArguments with no value type specified in iterable type array.
|
|
Continuous Integration / checks (8.3, true):
src/CssLint/Cli.php#L38
Parameter #1 $aOptions of method CssLint\Properties::setOptions() expects array, mixed given.
|
|
Continuous Integration / checks (8.3, true):
src/CssLint/Cli.php#L61
Method CssLint\Cli::parseArguments() has parameter $aArguments with no value type specified in iterable type array.
|
|
Continuous Integration / checks (8.3, true):
src/CssLint/Cli.php#L69
Method CssLint\Cli::printUsage() has no return type specified.
|
|
Continuous Integration / checks (8.3, true):
src/CssLint/Cli.php#L123
Parameter #1 $aErrors of method CssLint\Cli::displayLinterErrors() expects array, array|null given.
|
|
Continuous Integration / checks (8.3, true):
src/CssLint/Cli.php#L144
Parameter #1 $aErrors of method CssLint\Cli::displayLinterErrors() expects array, array|null given.
|
|
Continuous Integration / checks (8.3, true):
src/CssLint/Cli.php#L152
Method CssLint\Cli::printError() has no return type specified.
|
|
Continuous Integration / checks (8.1)
Failed to save: Failed to CreateCacheEntry: Received non-retryable error: Failed request: (409) Conflict: cache entry with the same key, version, and scope already exists
|