Add PHPCS. #42
puresyntax71Annotations
5 errors and 13 warnings
|
phpcs:
tests/src/FunctionalJavascript/Views/CivicrmContactUserRelationshipTest.php#L20
Do not disable strict config schema checking in tests. Instead ensure your module properly declares its schema for configurations.
|
|
phpcs:
tests/src/FunctionalJavascript/Views/ActivityFullcalendarViewTest.php#L23
Do not disable strict config schema checking in tests. Instead ensure your module properly declares its schema for configurations.
|
|
phpcs:
src/Access/ContactChecksumCheckAccess.php#L60
unserialize() is insecure unless allowed classes are limited. Use a safe format like JSON or use the allowed_classes option.
|
|
phpcs:
src/CiviEntityStorage.php#L597
unserialize() is insecure unless allowed classes are limited. Use a safe format like JSON or use the allowed_classes option.
|
|
phpcs
Process completed with exit code 1.
|
|
phpcs
Your workflow is using a version of actions/cache that is scheduled for deprecation, actions/cache@v2. Please update your workflow to use either v3 or v4 of actions/cache to avoid interruptions. Learn more: https://github.blog/changelog/2024-12-05-notice-of-upcoming-releases-and-breaking-changes-for-github-actions/#actions-cache-v1-v2-and-actions-toolkit-cache-package-closing-down
|
|
phpcs
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
|
phpcs:
tests/modules/civicrm_entity_test/civicrm_entity_test.info.yml#L7
All dependencies must be prefixed with the project name, for example "drupal:"
|
|
phpcs:
tests/modules/civicrm_entity_test/civicrm_entity_test.info.yml#L8
All dependencies must be prefixed with the project name, for example "drupal:"
|
|
phpcs:
tests/src/FunctionalJavascript/Views/ActivityFullcalendarViewTest.php#L95
Unused variable $fullcalendar.
|
|
phpcs:
civicrm_entity.info.yml#L7
All dependencies must be prefixed with the project name, for example "drupal:"
|
|
phpcs:
src/CivicrmEntityViewsData.php#L158
t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
|
|
phpcs:
src/CiviEntityStorage.php#L843
Unused variable $delta.
|
|
phpcs:
src/Form/CivicrmEntityForm.php#L79
\Drupal calls should be avoided in classes, use dependency injection instead
|
|
phpcs:
src/Plugin/Action/CivicrmContactAddToGroup.php#L135
t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
|
|
phpcs:
src/Plugin/Condition/DrupalUserExist.php#L101
User::load calls should be avoided in classes, use dependency injection instead
|
|
phpcs:
src/Plugin/Condition/DrupalUserExistsContactId.php#L100
User::load calls should be avoided in classes, use dependency injection instead
|
|
phpcs
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/
|