Merge pull request #358 from Nosto/release/4.3.0 #386
Annotations
15 warnings
|
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/cache@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/cache@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
classes/blocks/NostoDefaultTagging.php#L29
The class NostoDefaultTagging has a coupling between objects value of 16. Consider to reduce the number of dependencies under 13.
|
|
classes/blocks/NostoRecommendationElement.php#L40
Avoid variables with short names like $id. Configured minimum length is 3.
|
|
classes/helpers/NostoHelperConfig.php#L29
The class NostoHelperConfig has 17 public methods. Consider refactoring NostoHelperConfig to keep number of public methods under 10.
|
|
classes/helpers/NostoHelperConfig.php#L350
The 'getSkuEnabled()' method which returns a boolean should be named 'is...()' or 'has...()'
|
|
classes/helpers/NostoHelperConfig.php#L441
The 'getVariationEnabled()' method which returns a boolean should be named 'is...()' or 'has...()'
|
|
classes/helpers/NostoHelperConfig.php#L459
The 'getVariationTaxRuleEnabled()' method which returns a boolean should be named 'is...()' or 'has...()'
|
|
classes/helpers/NostoHelperContext.php#L29
The class NostoHelperContext has a coupling between objects value of 16. Consider to reduce the number of dependencies under 13.
|
|
classes/helpers/NostoHelperContext.php#L32
Avoid excessively long variable names like $backupShopContextStack. Keep variable name length under 20.
|
|
classes/helpers/NostoHelperController.php#L73
The method resolveObject() has a Cyclomatic Complexity of 11. The configured cyclomatic complexity threshold is 10.
|
|
classes/helpers/NostoHelperController.php#L79
Avoid variables with short names like $id. Configured minimum length is 3.
|
|
|
|
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/
|
|
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "4.3.0-Nosto.-.Personalization.for.PrestaShop", "phdpcpd-xml-result", "phpcs-xml-result", "phpmd-xml-result".
Please update your workflow to use v4 of the artifact actions.
Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
|
This job succeeded
Loading