Skip to content

[Mhv-65064] Resolve Download Accessibility Findings MR pt1. #64111

[Mhv-65064] Resolve Download Accessibility Findings MR pt1.

[Mhv-65064] Resolve Download Accessibility Findings MR pt1. #64111

Triggered via pull request December 23, 2024 20:12
Status Success
Total duration 16m 52s
Artifacts 2

pull-request.yml

on: pull_request
Fetch Unit Test Stability Allow List
46s
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 27s
Linting (Files Changed)
Check Cross App Imports
15m 52s
Check Cross App Imports
Fit to window
Zoom out
Zoom in

Annotations

12 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/mhv-medical-records/components/DownloadRecords/DownloadFileType.jsx#L453
src/applications/mhv-medical-records/components/DownloadRecords/DownloadFileType.jsx:453:13:The <va-button> Web Component should be used instead of the button HTML element.
Linting (Files Changed): src/applications/mhv-medical-records/components/DownloadRecords/DownloadFileType.jsx#L453
src/applications/mhv-medical-records/components/DownloadRecords/DownloadFileType.jsx:453:13:Missing an explicit type attribute for button
Linting (Files Changed): src/applications/mhv-medical-records/components/DownloadRecords/DownloadFileType.jsx#L465
src/applications/mhv-medical-records/components/DownloadRecords/DownloadFileType.jsx:465:13:The <va-button> Web Component should be used instead of the button HTML element.
Linting (Files Changed): src/applications/mhv-medical-records/components/DownloadRecords/DownloadFileType.jsx#L465
src/applications/mhv-medical-records/components/DownloadRecords/DownloadFileType.jsx:465:13:Missing an explicit type attribute for button
Linting (Files Changed): src/applications/mhv-medical-records/containers/DownloadReportPage.jsx#L118
src/applications/mhv-medical-records/containers/DownloadReportPage.jsx:118:67:React Hook useMemo has an unnecessary dependency: 'ccdError'. Either exclude it or remove the dependency array.
Linting (Files Changed): src/applications/mhv-medical-records/util/helpers.js#L1
src/applications/mhv-medical-records/util/helpers.js:1:1:Use date-fns or Native Date methods instead of moment.js
Linting (Files Changed): src/applications/mhv-medical-records/util/helpers.js#L263
src/applications/mhv-medical-records/util/helpers.js:263:66:Consider using date-fns format(date, format) or dayjs().format()
Linting (Files Changed): src/applications/mhv-medical-records/util/helpers.js#L263
src/applications/mhv-medical-records/util/helpers.js:263:66:Consider using Native new Date().
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