Skip to content

VACMS - 19549 19800 conditional render services select, and loading indicator until CCP services loaded #65016

VACMS - 19549 19800 conditional render services select, and loading indicator until CCP services loaded

VACMS - 19549 19800 conditional render services select, and loading indicator until CCP services loaded #65016

Triggered via pull request January 7, 2025 23:22
Status Success
Total duration 16m 28s
Artifacts 2

pull-request.yml

on: pull_request
Fetch Unit Test Stability Allow List
45s
Fetch Unit Test Stability Allow List
Check Cross App Imports
15m 25s
Check Cross App Imports
Fit to window
Zoom out
Zoom in

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