Skip to content

Merge pull request #361 from Nosto/division-by-zero-when-tax-is-zero #399

Merge pull request #361 from Nosto/division-by-zero-when-tax-is-zero

Merge pull request #361 from Nosto/division-by-zero-when-tax-is-zero #399

Triggered via push September 17, 2024 18:26
Status Success
Total duration 38s
Artifacts 4

action.yml

on: push
Code Sniffer
24s
Code Sniffer
Mess Detect
18s
Mess Detect
Copy-Paste Detect
30s
Copy-Paste Detect
Package
21s
Package
Fit to window
Zoom out
Zoom in

Annotations

27 warnings
Mess Detect
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/
Mess Detect
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/cache@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Mess Detect: 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.
Mess Detect: classes/blocks/NostoRecommendationElement.php#L40
Avoid variables with short names like $id. Configured minimum length is 3.
Mess Detect: classes/helpers/NostoHelperConfig.php#L29
The class NostoHelperConfig has 17 public methods. Consider refactoring NostoHelperConfig to keep number of public methods under 10.
Mess Detect: classes/helpers/NostoHelperConfig.php#L350
The 'getSkuEnabled()' method which returns a boolean should be named 'is...()' or 'has...()'
Mess Detect: classes/helpers/NostoHelperConfig.php#L441
The 'getVariationEnabled()' method which returns a boolean should be named 'is...()' or 'has...()'
Mess Detect: classes/helpers/NostoHelperConfig.php#L459
The 'getVariationTaxRuleEnabled()' method which returns a boolean should be named 'is...()' or 'has...()'
Mess Detect: 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.
Mess Detect: classes/helpers/NostoHelperContext.php#L32
Avoid excessively long variable names like $backupShopContextStack. Keep variable name length under 20.
Mess Detect: classes/helpers/NostoHelperController.php#L73
The method resolveObject() has a Cyclomatic Complexity of 11. The configured cyclomatic complexity threshold is 10.
Mess Detect: classes/helpers/NostoHelperController.php#L79
Avoid variables with short names like $id. Configured minimum length is 3.
Mess Detect
Skipping prestissimo, as it does not support Composer . Specify composer:v1 in tools to use prestissimo
Mess Detect
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/
Package
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/
Package
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/cache@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Package
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/
Package
Skipping prestissimo, as it does not support Composer . Specify composer:v1 in tools to use prestissimo
Package
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/
Code Sniffer
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/
Code Sniffer
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/cache@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Code Sniffer
Skipping prestissimo, as it does not support Composer . Specify composer:v1 in tools to use prestissimo
Code Sniffer
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/
Copy-Paste Detect
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/
Copy-Paste Detect
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/cache@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Copy-Paste Detect
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/
Copy-Paste Detect
Skipping prestissimo, as it does not support Composer . Specify composer:v1 in tools to use prestissimo

Artifacts

Produced during runtime
Name Size
4.4.1-Nosto.-.Personalization.for.PrestaShop Expired
1.27 MB
phdpcpd-xml-result Expired
188 Bytes
phpcs-xml-result Expired
207 Bytes
phpmd-xml-result Expired
3.32 KB