Skip to content

Replace super-linter with a reusable workflow #204

Replace super-linter with a reusable workflow

Replace super-linter with a reusable workflow #204

Triggered via pull request May 13, 2025 17:46
Status Failure
Total duration 2m 16s
Artifacts

php.yml

on: pull_request
Matrix: PHP-Linter
Linter  /  ecmascript-linter
Linter / ecmascript-linter
Linter  /  json-linter
Linter / json-linter
Linter  /  style-linter
Linter / style-linter
Linter  /  yaml-linter
Linter / yaml-linter
Matrix: unit-tests-linux
Matrix: unit-tests-windows
Code coverage
7s
Code coverage
Quality control
51s
Quality control
Security checks
36s
Security checks
Cleanup artifacts
4s
Cleanup artifacts
Fit to window
Zoom out
Zoom in

Annotations

21 errors and 3 warnings
UnusedVariable: tests/src/test-metadata/saml20-idp-remote.php#L3
tests/src/test-metadata/saml20-idp-remote.php:3:1: UnusedVariable: $metadata is never referenced or the value is not used (see https://psalm.dev/077)
UnusedMethodCall: tests/src/PowerIdPDiscoTest.php#L82
tests/src/PowerIdPDiscoTest.php:82:29: UnusedMethodCall: The call to ReflectionMethod::setAccessible is not used (see https://psalm.dev/209)
UnusedMethodCall: tests/src/PowerIdPDiscoTest.php#L70
tests/src/PowerIdPDiscoTest.php:70:22: UnusedMethodCall: The call to ReflectionMethod::setAccessible is not used (see https://psalm.dev/209)
MissingOverrideAttribute: tests/src/PowerIdPDiscoTest.php#L26
tests/src/PowerIdPDiscoTest.php:26:5: MissingOverrideAttribute: Method SimpleSAML\Test\Module\discopower\PowerIdPDiscoTest::setupbeforeclass should have the "Override" attribute (see https://psalm.dev/358)
UnusedClass: tests/src/PowerIdPDiscoTest.php#L15
tests/src/PowerIdPDiscoTest.php:15:7: UnusedClass: Class SimpleSAML\Test\Module\discopower\PowerIdPDiscoTest is never used (see https://psalm.dev/075)
UnusedVariable: tests/src/Controller/DiscoPowerTest.php#L182
tests/src/Controller/DiscoPowerTest.php:182:9: UnusedVariable: $r is never referenced or the value is not used (see https://psalm.dev/077)
UnusedVariable: tests/src/Controller/DiscoPowerTest.php#L162
tests/src/Controller/DiscoPowerTest.php:162:9: UnusedVariable: $r is never referenced or the value is not used (see https://psalm.dev/077)
UnusedVariable: tests/src/Controller/DiscoPowerTest.php#L64
tests/src/Controller/DiscoPowerTest.php:64:9: UnusedVariable: $r is never referenced or the value is not used (see https://psalm.dev/077)
MissingOverrideAttribute: tests/src/Controller/DiscoPowerTest.php#L28
tests/src/Controller/DiscoPowerTest.php:28:5: MissingOverrideAttribute: Method SimpleSAML\Test\Module\discopower\Controller\DiscoPowerTest::setup should have the "Override" attribute (see https://psalm.dev/358)
UnusedClass: tests/src/Controller/DiscoPowerTest.php#L19
tests/src/Controller/DiscoPowerTest.php:19:7: UnusedClass: Class SimpleSAML\Test\Module\discopower\Controller\DiscoPowerTest is never used (see https://psalm.dev/075)
MissingOverrideAttribute: src/PowerIdPDisco.php#L220
src/PowerIdPDisco.php:220:5: MissingOverrideAttribute: Method SimpleSAML\Module\discopower\PowerIdPDisco::filterlist should have the "Override" attribute (see https://psalm.dev/358)
UnusedForeachValue: src/PowerIdPDisco.php#L168
src/PowerIdPDisco.php:168:36: UnusedForeachValue: $tbslist is never referenced or the value is not used (see https://psalm.dev/275)
PossiblyUnusedReturnValue: src/PowerIdPDisco.php#L108
src/PowerIdPDisco.php:108:16: PossiblyUnusedReturnValue: The return value for this method is never used (see https://psalm.dev/273)
MissingOverrideAttribute: src/PowerIdPDisco.php#L91
src/PowerIdPDisco.php:91:5: MissingOverrideAttribute: Method SimpleSAML\Module\discopower\PowerIdPDisco::log should have the "Override" attribute (see https://psalm.dev/358)
ClassMustBeFinal: src/PowerIdPDisco.php#L26
src/PowerIdPDisco.php:26:7: ClassMustBeFinal: Class SimpleSAML\Module\discopower\PowerIdPDisco is never extended and is not part of the public API, and thus must be made final. (see https://psalm.dev/361)
ArgumentTypeCoercion: src/Controller/DiscoPower.php#L64
src/Controller/DiscoPower.php:64:36: ArgumentTypeCoercion: Argument 1 of Symfony\Component\HttpFoundation\JsonResponse::setCallback expects null|string, but parent type null|scalar provided (see https://psalm.dev/193)
PossiblyNullArgument: src/Controller/DiscoPower.php#L61
src/Controller/DiscoPower.php:61:48: PossiblyNullArgument: Argument 2 of preg_match cannot be null, possibly null value provided (see https://psalm.dev/078)
ArgumentTypeCoercion: src/Controller/DiscoPower.php#L61
src/Controller/DiscoPower.php:61:48: ArgumentTypeCoercion: Argument 2 of preg_match expects string, but parent type null|scalar provided (see https://psalm.dev/193)
PossiblyUnusedParam: src/Controller/DiscoPower.php#L19
src/Controller/DiscoPower.php:19:34: PossiblyUnusedParam: Param #1 is never referenced in this method (see https://psalm.dev/134)
ClassMustBeFinal: src/Controller/DiscoPower.php#L13
src/Controller/DiscoPower.php:13:7: ClassMustBeFinal: Class SimpleSAML\Module\discopower\Controller\DiscoPower is never extended and is not part of the public API, and thus must be made final. (see https://psalm.dev/361)
Unit tests, PHP 8.4, windows-latest
Process completed with exit code 1.
ClassMustBeFinal: tests/src/PowerIdPDiscoTest.php#L15
tests/src/PowerIdPDiscoTest.php:15:7: ClassMustBeFinal: Class SimpleSAML\Test\Module\discopower\PowerIdPDiscoTest is never extended and is not part of the public API, and thus must be made final. (see https://psalm.dev/361)
ClassMustBeFinal: tests/src/Controller/DiscoPowerTest.php#L19
tests/src/Controller/DiscoPowerTest.php:19:7: ClassMustBeFinal: Class SimpleSAML\Test\Module\discopower\Controller\DiscoPowerTest is never extended and is not part of the public API, and thus must be made final. (see https://psalm.dev/361)
UnusedClass: src/Controller/DiscoPower.php#L13
src/Controller/DiscoPower.php:13:7: UnusedClass: Class SimpleSAML\Module\discopower\Controller\DiscoPower is never used (see https://psalm.dev/075)