Skip to content

VACMS 19548 - orientation of map and controls for Facility Locator #64124

VACMS 19548 - orientation of map and controls for Facility Locator

VACMS 19548 - orientation of map and controls for Facility Locator #64124

Triggered via pull request December 23, 2024 23:37
Status Success
Total duration 17m 59s
Artifacts 2

pull-request.yml

on: pull_request
Fetch Unit Test Stability Allow List
50s
Fetch Unit Test Stability Allow List
ESLint Disable Check
0s
ESLint Disable Check
Icon Check
0s
Icon Check
Sentry Check
0s
Sentry Check
Linting (Files Changed)
1m 24s
Linting (Files Changed)
Check Cross App Imports
16m 52s
Check Cross App Imports
Fit to window
Zoom out
Zoom in

Annotations

14 warnings
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#L172
src/applications/facility-locator/components/SearchControls.jsx:172: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#L201
src/applications/facility-locator/components/SearchControls.jsx:201: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/map-zoom.cypress.spec.js#L9
src/applications/facility-locator/tests/e2e/map-zoom.cypress.spec.js:9:5:It is unsafe to chain further commands that rely on the subject after this command. It is best to split the chain, chaining again from `cy.` in a next command line.
Linting (Files Changed): src/applications/facility-locator/tests/e2e/map-zoom.cypress.spec.js#L21
src/applications/facility-locator/tests/e2e/map-zoom.cypress.spec.js:21:5:It is unsafe to chain further commands that rely on the subject after this command. It is best to split the chain, chaining again from `cy.` in a next command line.
Linting (Files Changed): src/applications/facility-locator/tests/e2e/map-zoom.cypress.spec.js#L33
src/applications/facility-locator/tests/e2e/map-zoom.cypress.spec.js:33:5:It is unsafe to chain further commands that rely on the subject after this command. It is best to split the chain, chaining again from `cy.` in a next command line.
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