Cleanup, add PHP file support #164
Annotations
10 errors and 11 warnings
Lint PHP files:
inc/WebhookHandler.php#L23
Incorrect order of annotations groups.
|
Lint PHP files:
inc/WebhookHandlerFactory.php#L24
There must be a single blank line after a tag group
|
Lint PHP files:
inc/WebhookHandlerFactory.php#L27
Expected 1 line between annotations groups, found 2.
|
Lint PHP files:
inc/CLI/InfoCommand.php#L152
Function is_string() should be referenced via a fully qualified name.
|
Lint PHP files:
tests/phpunit/tests/RestrictedSiteAccess.php#L5
Use of annotation @Package is forbidden.
|
Lint PHP files:
tests/phpunit/tests/RestrictedSiteAccess.php#L16
Missing doc comment for function test_does_not_change_restriction_by_default()
|
Lint PHP files:
tests/phpunit/tests/RestrictedSiteAccess.php#L28
Missing doc comment for function test_lifts_restriction_for_gp_translations_api_route()
|
Lint PHP files:
tests/phpunit/tests/RestrictedSiteAccess.php#L40
Missing doc comment for function test_does_not_change_restriction_for_other_gp_api_route()
|
Lint PHP files:
tests/phpunit/tests/RestrictedSiteAccess.php#L52
Missing doc comment for function test_lifts_restriction_for_incoming_webhook_rest_route()
|
Lint PHP files:
tests/phpunit/tests/RestrictedSiteAccess.php#L64
Missing doc comment for function test_lifts_restriction_for_legacy_incoming_webhook_rest_rout()
|
Get Composer home directory
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Lint PHP files:
tests/phpunit/bootstrap.php#L1
A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 54 and the first side effect is on line 8.
|
Lint PHP files:
tests/phpunit/bootstrap.php#L28
putenv() found. Changing configuration values at runtime is strongly discouraged.
|
Lint PHP files:
tests/phpunit/tests/Updater.php#L105
serialize() found. Serialized data has known vulnerability problems with Object Injection. JSON is generally a better approach for serializing data. See https://www.owasp.org/index.php/PHP_Object_Injection
|
Lint PHP files:
tests/phpunit/tests/Updater.php#L110
Unused variable $timestamp.
|
Lint PHP files:
tests/phpunit/tests/Updater.php#L122
Unused variable $timestamp.
|
Lint PHP files:
tests/phpunit/tests/Updater.php#L136
Unused variable $timestamp.
|
Lint PHP files:
tests/phpunit/tests/ZipProvider.php#L315
Equals sign not aligned correctly; expected 1 space but found 2 spaces
|
Lint PHP files:
tests/phpunit/tests/ZipProvider.php#L335
Found precision alignment of 2 spaces.
|
Lint PHP files:
tests/phpunit/tests/ZipProvider.php#L336
Found precision alignment of 2 spaces.
|
Lint PHP files:
tests/phpunit/tests/ZipProvider.php#L337
Found precision alignment of 2 spaces.
|
Loading