PHP CI

Merge branch 'develop' into partial-product-sync #1062

Triggered via push September 17, 2024 09:07

supercid

pushed 9c9121e

partial-product-sync

Status Failure

Total duration 43s

Artifacts 3

action.yml

on: push

Annotations

5 errors and 29 warnings

Code Sniffer: Model/Product/Partial/PriceProduct.php#L83

PHP syntax error: syntax error, unexpected ')', expecting variable (T_VARIABLE)

Code Sniffer: Model/Product/Partial/InventoryProduct.php#L79

PHP syntax error: syntax error, unexpected ')', expecting variable (T_VARIABLE)

Code Sniffer: Model/Indexer/Partial/ProductInventoryIndexer.php#L90

PHP syntax error: syntax error, unexpected ')', expecting variable (T_VARIABLE)

Code Sniffer: Model/Indexer/Partial/ProductPriceIndexer.php#L90

PHP syntax error: syntax error, unexpected ')', expecting variable (T_VARIABLE)

Code Sniffer

Process completed with exit code 1.

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: Model/Service/Product/Partial/InventoryProductService.php#L89

Comment refers to a TODO task "Can be better abstracted"

Code Sniffer: Model/Service/Product/Partial/PriceProductService.php#L89

Comment refers to a TODO task "Can be better abstracted"

Code Sniffer: Model/Service/Sync/Partial/PriceBulkConsumer.php#L55

Class description should contain additional information beyond the name already supplies.

Code Sniffer: Model/Service/Sync/Partial/InventoryBulkConsumer.php#L55

Class description should contain additional information beyond the name already supplies.

Code Sniffer: Model/Service/Sync/Partial/PriceService.php#L106

Multi-line function declaration not indented correctly; expected 8 spaces but found 27

Code Sniffer: Model/Service/Sync/Partial/PriceService.php#L107

Multi-line function declaration not indented correctly; expected 8 spaces but found 27

Code Sniffer: Model/Product/Partial/PriceProduct.php#L66

Code must not contain multiple empty lines in a row; found 3 empty lines.

Code Sniffer: Model/Product/Partial/PriceProduct.php#L128

Expected 1 newline at end of file; 0 found

Code Sniffer: Model/Indexer/Partial/ProductInventoryIndexer.php#L145

Expected 1 newline at end of file; 0 found

Code Sniffer: Model/Indexer/Partial/ProductPriceIndexer.php#L145

Expected 1 newline at end of file; 0 found

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/

Mess Detect

Mess Detect: Block/Adminhtml/Form/Field/Tokens.php#L87

Avoid variables with short names like $id. Configured minimum length is 3.

Mess Detect: Block/Product.php#L63

The class Product has a coupling between objects value of 18. Consider to reduce the number of dependencies under 13.

Mess Detect: Block/TaggingTrait.php#L68

The method _toHtml is not named in camelCase.

Mess Detect: Console/Command/NostoAccountConnectCommand.php#L142