(BSR)[PRO] refactor: miscellaneous new archi #7701
rchaffal-passdev_on_pull_request_workflow.yml
on: pull_request
Init job
40s
Tests pro
/
...
/
Style quality check
1m 35s
Tests pro
/
Unit tests
8m 41s
[pcapi-tests] build docker image.
/
build-docker-image
[pcapi-console] build docker image.
/
Build pcapi-console:0b807b895ce322e209487d702b8f8589fbd0f495
3m 38s
MyPy cop
/
MyPy cop test
Tests API documentation
/
Check API documentation can be built
[PRO] Deploy PR version for validation
/
deploy_on_firebase
1m 25s
Dependabot
/
dependabot
Matrix: Tests api / Pytest
Waiting for pending jobs
Tests api
/
Pylint
Tests api
/
Quality checks
Tests api
/
Test database schema
Update api client template
/
Update API Client template
Push pcapi docker image to registry
/
push-docker-image
Push pcapi-console docker image to registry
/
push-docker-image
Annotations
10 warnings and 2 notices
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/app/App/App.tsx#L74
React Hook useEffect has a missing dependency: 'dispatch'. Either include it or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/app/App/hook/useLogExtraProData.ts#L25
React Hook useEffect has missing dependencies: 'location.pathname', 'logEvent', and 'previousOfferer'. Either include them or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/app/App/hook/useLogNavigation.ts#L32
React Hook useEffect has missing dependencies: 'params' and 'previousPage'. Either include them or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/app/App/hook/usePageTitle.ts#L16
React Hook useEffect has a missing dependency: 'location'. Either include it or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/commons/hooks/useGetBitmap.ts#L21
React Hook useEffect has a missing dependency: 'file'. Either include it or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/commons/hooks/useIsElementVisible.ts#L18
The 'observer' object construction makes the dependencies of useEffect Hook (at line 32) change on every render. Move it inside the useEffect callback. Alternatively, wrap the initialization of 'observer' in its own useMemo() Hook
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/commons/hooks/useNotification.ts#L50
React Hook useMemo has a missing dependency: 'notify'. Either include it or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/commons/hooks/useRedirectLoggedUser.ts#L53
React Hook useEffect has missing dependencies: 'dispatch', 'navigate', and 'redirectToUrl'. Either include them or remove the dependency array
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/commons/utils/sortCollectiveOffers.ts#L19
Switch is not exhaustive. Cases not matched: null
|
|
Tests pro / Type check / Quality check / Style quality check:
pro/src/commons/utils/timezone.ts#L56
Switch is not exhaustive. Cases not matched: undefined | null
|
|
[PRO] Deploy PR version for validation / deploy_on_firebase
Firebase deployment is available at : https://pc-pro-testing--pr15206-bsr-components-migra-y8kt1tt4.web.app
|
|
Tests pro E2E / E2E tests and notifications
Running e2e-tests with europe-west1-docker.pkg.dev/passculture-infra-prod/pass-culture-artifact-registry/pcapi:latest
|
Artifacts
Produced during runtime
| Name | Size | |
|---|---|---|
|
pass-culture~pass-culture-main~W3S8NJ.dockerbuild
|
94.5 KB |
|
|
pcapi-console-0b807b895ce322e209487d702b8f8589fbd0f495.tar
Expired
|
482 MB |
|