Skip to content

Change logic of the useVIewCartPixel hook #461

Change logic of the useVIewCartPixel hook

Change logic of the useVIewCartPixel hook #461

Triggered via pull request August 2, 2024 08:16
Status Failure
Total duration 1m 14s
Artifacts

pull-request.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 8 warnings
Lint: react/Minicart.tsx#L69
Replace `variation·===·'drawer'·?·isDrawerOpen·:·open,·orderForm?.items` with `⏎····variation·===·'drawer'·?·isDrawerOpen·:·open,⏎····orderForm?.items⏎··`
Lint: react/Minicart.tsx#L107
React Hook "useCallback" is called conditionally. React Hooks must be called in the exact same order in every component render. Did you accidentally call a React Hook after an early return?
Lint: react/Minicart.tsx#L107
Insert `⏎····`
Lint: react/Minicart.tsx#L108
Insert `··`
Lint: react/Minicart.tsx#L109
Replace `··},·[setIsDrawerOpen]` with `····},⏎····[setIsDrawerOpen]⏎··`
Lint: react/modules/useViewCartPixel.tsx#L4
There should be at least one empty line between import groups
Lint: react/modules/useViewCartPixel.tsx#L5
`debounce` import should occur before import of `./pixelHelper`
Lint: react/modules/useViewCartPixel.tsx#L20
'pushViewCart' was used before it was defined
Lint: react/modules/useViewCartPixel.tsx#L21
React Hook useEffect has a missing dependency: 'pushViewCart'. Either include it or remove the dependency array
Lint: react/modules/useViewCartPixel.tsx#L27
Expected property shorthand
IO app test
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/
IO app test
The following actions uses Node.js version which is deprecated 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/
IO app test
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/
Lint
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/
Lint
The following actions uses Node.js version which is deprecated 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/
Lint
Docker build failed with exit code 1, back off 4.087 seconds before retry.
Lint: react/typings/vtex.store-drawer.d.ts#L2
Unexpected any. Specify a different type
Lint
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/