Skip to content

clean code.

clean code. #4182

Triggered via push December 4, 2024 12:50
Status Failure
Total duration 1m 7s
Artifacts

test.yml

on: push
Basic tests
58s
Basic tests
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 11 warnings
Basic tests: src/navigation/courier/barcode/Barcode.js#L172
The 'checkClientAction' function makes the dependencies of useEffect Hook (at line 222) change on every render. Move it inside the useEffect callback. Alternatively, wrap the definition of 'checkClientAction' in its own useCallback() Hook
Basic tests
Process completed with exit code 1.
Basic tests
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Basic tests: src/API.js#L358
Promise executor functions should not be async
Basic tests: src/API.js#L501
Do not access Object.prototype method 'hasOwnProperty' from target object
Basic tests: src/API.js#L580
'refreshToken' is already declared in the upper scope on line 580 column 5
Basic tests: src/API.js#L699
Do not access Object.prototype method 'hasOwnProperty' from target object
Basic tests: src/API.js#L735
Do not access Object.prototype method 'hasOwnProperty' from target object
Basic tests: src/API.js#L736
Do not access Object.prototype method 'hasOwnProperty' from target object
Basic tests: src/API.js#L737
Do not access Object.prototype method 'hasOwnProperty' from target object
Basic tests: src/AppUser.js#L95
Do not access Object.prototype method 'hasOwnProperty' from target object
Basic tests: src/AppUser.js#L98
Do not access Object.prototype method 'hasOwnProperty' from target object
Basic tests: src/__tests__/API.test.js#L120
Disabled test