Multiplier: Add missing param typehint to getValues #54
codesniffer.yml
on: pull_request
Codesniffer
/
Codesniffer (8.1)
25s
Annotations
8 errors and 1 warning
Codesniffer / Codesniffer (8.1):
src/Multiplier.php#L69
Property \Contributte\FormMultiplier\Multiplier::$resolver has useless @var annotation.
|
Codesniffer / Codesniffer (8.1):
src/Multiplier.php#L70
The placement of "protected properties" group is invalid. Last group was "private properties" and one of these is expected after it: private properties, constructor, destructor, static constructors, public static abstract methods, public static final methods, public static methods, public abstract methods, public final methods, public methods, protected static abstract methods, protected static final methods, protected static methods, protected abstract methods, protected final methods, protected methods, private static methods, private methods, magic methods
|
Codesniffer / Codesniffer (8.1):
src/Multiplier.php#L216
Expected 1 line after "if", found 0.
|
Codesniffer / Codesniffer (8.1):
src/Multiplier.php#L284
Method \Contributte\FormMultiplier\Multiplier::getValues() does not have native type hint for its parameter $returnType but it should be possible to add it based on @param annotation "string|object|bool|null".
|
Codesniffer / Codesniffer (8.1):
src/Multiplier.php#L344
Tabs must be used to indent lines; spaces are not allowed
|
Codesniffer / Codesniffer (8.1):
src/Multiplier.php#L392
Tabs must be used to indent lines; spaces are not allowed
|
Codesniffer / Codesniffer (8.1):
src/Multiplier.php#L393
Tabs must be used to indent lines; spaces are not allowed
|
Codesniffer / Codesniffer (8.1)
Process completed with exit code 2.
|
Codesniffer / Codesniffer (8.1)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|