VACMS - 19549 19800 conditional render services select, and loading indicator until CCP services loaded #65016
pull-request.yml
on: pull_request
Fetch Unit Test Stability Allow List
45s
ESLint Disable Check
1m 15s
Icon Check
1m 19s
Sentry Check
1m 16s
Linting (Files Changed)
1m 25s
Check Cross App Imports
15m 25s
Annotations
17 warnings
Sentry Check
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
ESLint Disable Check
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
Icon Check
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
Linting (Files Changed)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
Linting (Files Changed)
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/
|
Linting (Files Changed)
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/
|
Linting (Files Changed):
src/applications/facility-locator/components/SearchControls.jsx#L170
src/applications/facility-locator/components/SearchControls.jsx:170:13:The <va-button> Web Component should be used instead of the button HTML element.
|
Linting (Files Changed):
src/applications/facility-locator/components/SearchControls.jsx#L198
src/applications/facility-locator/components/SearchControls.jsx:198:13:The <va-button> Web Component should be used instead of the button HTML element.
|
Linting (Files Changed):
src/applications/facility-locator/containers/FacilitiesMap.jsx#L659
src/applications/facility-locator/containers/FacilitiesMap.jsx:659:55:The ref value 'mapboxContainerRef.current' will likely have changed by the time this effect cleanup function runs. If this ref points to a node rendered by React, copy 'mapboxContainerRef.current' to a variable inside the effect, and use that variable in the cleanup function.
|
Linting (Files Changed):
src/applications/facility-locator/containers/FacilitiesMap.jsx#L695
src/applications/facility-locator/containers/FacilitiesMap.jsx:695:5:React Hook useEffect has a missing dependency: 'searchCurrentArea'. Either include it or remove the dependency array.
|
Linting (Files Changed):
src/applications/facility-locator/containers/FacilitiesMap.jsx#L708
src/applications/facility-locator/containers/FacilitiesMap.jsx:708:5:React Hook useEffect has a missing dependency: 'handleSearchOnQueryChange'. Either include it or remove the dependency array.
|
Linting (Files Changed):
src/applications/facility-locator/containers/FacilitiesMap.jsx#L716
src/applications/facility-locator/containers/FacilitiesMap.jsx:716:5:React Hook useEffect has a missing dependency: 'renderMarkers'. Either include it or remove the dependency array.
|
Linting (Files Changed):
src/applications/facility-locator/containers/FacilitiesMap.jsx#L732
src/applications/facility-locator/containers/FacilitiesMap.jsx:732:5:React Hook useEffect has a missing dependency: 'handleMapOnNoResultsFound'. Either include it or remove the dependency array.
|
Linting (Files Changed):
src/applications/facility-locator/tests/e2e/errorMessages.cypress.spec.js#L24
src/applications/facility-locator/tests/e2e/errorMessages.cypress.spec.js:24:3:Cypress E2E tests must include at least one axeCheck call. Documentation for adding checks and understanding errors can be found here: https://depo-platform-documentation.scrollhelp.site/developer-docs/A11y-Testing.1935409178.html
|
Linting (Files Changed):
src/applications/facility-locator/tests/e2e/errorMessages.cypress.spec.js#L32
src/applications/facility-locator/tests/e2e/errorMessages.cypress.spec.js:32:3:Cypress E2E tests must include at least one axeCheck call. Documentation for adding checks and understanding errors can be found here: https://depo-platform-documentation.scrollhelp.site/developer-docs/A11y-Testing.1935409178.html
|
Linting (Files Changed):
src/applications/facility-locator/tests/e2e/errorMessages.cypress.spec.js#L45
src/applications/facility-locator/tests/e2e/errorMessages.cypress.spec.js:45:3:Cypress E2E tests must include at least one axeCheck call. Documentation for adding checks and understanding errors can be found here: https://depo-platform-documentation.scrollhelp.site/developer-docs/A11y-Testing.1935409178.html
|
Check Cross App Imports
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
Artifacts
Produced during runtime
Name | Size | |
---|---|---|
cross-app-import-report
|
2.78 KB |
|
unit-test-allow-list
|
32.7 KB |
|