Skip to content

Mhv 65291 add pending meds to list #67207

Mhv 65291 add pending meds to list

Mhv 65291 add pending meds to list #67207

Triggered via pull request January 31, 2025 22:06
Status Success
Total duration 16m 54s
Artifacts 2

pull-request.yml

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

Annotations

12 warnings
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/mhv-medications/components/PrescriptionDetails/VaPrescription.jsx#L70
src/applications/mhv-medications/components/PrescriptionDetails/VaPrescription.jsx:70:22:Refactor this function to reduce its Cognitive Complexity from 93 to the 50 allowed.
Linting (Files Changed): src/applications/mhv-medications/containers/Prescriptions.jsx#L206
src/applications/mhv-medications/containers/Prescriptions.jsx:206:6:React Hook useEffect has missing dependencies: 'isLoading' and 'prescriptionId'. Either include them or remove the dependency array.
Linting (Files Changed): src/applications/mhv-medications/containers/Prescriptions.jsx#L296
src/applications/mhv-medications/containers/Prescriptions.jsx:296:5:React Hook useEffect has a missing dependency: 'showFilterContent'. Either include it or remove the dependency array.
Linting (Files Changed): src/applications/mhv-medications/containers/Prescriptions.jsx#L306
src/applications/mhv-medications/containers/Prescriptions.jsx:306:6:React Hook useEffect has a missing dependency: 'filterOption'. Either include it or remove the dependency array.
Linting (Files Changed): src/applications/mhv-medications/containers/Prescriptions.jsx#L330
src/applications/mhv-medications/containers/Prescriptions.jsx:330:5:React Hook useEffect has missing dependencies: 'selectedSortOption' and 'showGroupingContent'. Either include them or remove the dependency array.
Linting (Files Changed): src/applications/mhv-medications/containers/Prescriptions.jsx#L416
src/applications/mhv-medications/containers/Prescriptions.jsx:416:5:React Hook useCallback has a missing dependency: 'showFilterContent'. Either include it or remove the dependency array.
Linting (Files Changed): src/applications/mhv-medications/containers/Prescriptions.jsx#L442
src/applications/mhv-medications/containers/Prescriptions.jsx:442:5:React Hook useCallback has a missing dependency: 'showFilterContent'. Either include it or remove the dependency array.
Linting (Files Changed): src/applications/mhv-medications/containers/Prescriptions.jsx#L562
src/applications/mhv-medications/containers/Prescriptions.jsx:562:5:React Hook useEffect has missing dependencies: 'dispatch', 'filteredList', 'paginatedPrescriptionsList', 'printRxList', 'selectedSortOption', and 'showFilterContent'. Either include them or remove the dependency array.
Linting (Files Changed): src/applications/mhv-medications/tests/e2e/pages/MedicationsListPage.js#L1
src/applications/mhv-medications/tests/e2e/pages/MedicationsListPage.js:1:1:Use date-fns or Native Date methods instead of moment.js
Linting (Files Changed): src/applications/mhv-medications/tests/e2e/pages/MedicationsListPage.js#L276
src/applications/mhv-medications/tests/e2e/pages/MedicationsListPage.js:276:29:Consider using date-fns addDays(date, amount) or dayjs().add(number, unit)

Artifacts

Produced during runtime
Name Size
cross-app-import-report
2.31 KB
unit-test-allow-list
32.7 KB