Skip to content
PHP_CodeSniffer

implement basic structure #4

Sign in to view logs

implement basic structure

implement basic structure #4

Triggered via pull request November 8, 2024 08:31
@bohdanzhubohdanzhu
synchronize #1
bohdanzhu:26804-basic-structure
Status Failure
Total duration 19s
Artifacts –

phpcs.yml

on: pull_request
PHP_CodeSniffer
11s
PHP_CodeSniffer
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 6 warnings
PHP_CodeSniffer: Civi/Notification/Entity/AbstractEntity.php#L22
Expected type hint "T"; found "array" for $entityValues
PHP_CodeSniffer: tests/phpunit/Civi/Notification/Source/ValueComparatorTest.php#L1
Missing required strict_types declaration
PHP_CodeSniffer: tests/phpunit/Civi/Notification/Source/ValueComparatorTest.php#L63
Expected 1 blank line after function; 0 found
PHP_CodeSniffer: tests/phpunit/Civi/Notification/Source/ValueComparatorTest.php#L64
The closing brace for the class must have an empty line before it
PHP_CodeSniffer: tests/phpunit/Civi/Notification/Handler/RuleSetHandlerTest.php#L1
Missing required strict_types declaration
PHP_CodeSniffer: tests/phpunit/Civi/Notification/Handler/RuleSetHandlerTest.php#L30
TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
PHP_CodeSniffer: tests/phpunit/Civi/Notification/Handler/RuleSetHandlerTest.php#L36
TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
PHP_CodeSniffer: tests/phpunit/Civi/Notification/Handler/RuleSetHandlerTest.php#L37
TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
PHP_CodeSniffer: tests/phpunit/Civi/Notification/Handler/RuleSetHandlerTest.php#L38
TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
PHP_CodeSniffer: tests/phpunit/Civi/Notification/Handler/RuleSetHandlerTest.php#L52
TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
PHP_CodeSniffer
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/cache@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
PHP_CodeSniffer: Civi/Notification/Interface/ContactLoaderInterface.php#L8
Unused use statement
PHP_CodeSniffer: Civi/Notification/Interface/MsgTemplateDeterminerInterface.php#L8
Unused use statement
PHP_CodeSniffer: Civi/Notification/Handler/RuleHandler.php#L7
Unused use statement
PHP_CodeSniffer: tests/phpunit/Civi/Notification/EventSubscriber/NotificationSubscriberTest.php#L57
This comment is 71% valid code; is this commented out code?
PHP_CodeSniffer: tests/phpunit/Civi/Notification/EventSubscriber/NotificationSubscriberTest.php#L82
This comment is 66% valid code; is this commented out code?