Skip to content

MHV-53461 reworked hide alert #57744

MHV-53461 reworked hide alert

MHV-53461 reworked hide alert #57744

Triggered via pull request October 4, 2024 21:25
Status Success
Total duration 8m 25s
Artifacts 1

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 33s
Linting (Files Changed)
Check Cross App Imports
7m 24s
Check Cross App Imports
Fit to window
Zoom out
Zoom in

Annotations

10 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-secure-messaging/actions/messages.js#L1
src/applications/mhv-secure-messaging/actions/messages.js:1:1:Use date-fns or Native Date methods instead of moment.js
Linting (Files Changed): src/applications/mhv-secure-messaging/actions/messages.js#L64
src/applications/mhv-secure-messaging/actions/messages.js:64:11:Consider using Native new Date().
Linting (Files Changed): src/applications/mhv-secure-messaging/components/AttachmentsList.jsx#L47
src/applications/mhv-secure-messaging/components/AttachmentsList.jsx:47:5:React Hook useEffect has a missing dependency: 'attachments.length'. Either include it or remove the dependency array.
Linting (Files Changed): src/applications/mhv-secure-messaging/components/AttachmentsList.jsx#L90
src/applications/mhv-secure-messaging/components/AttachmentsList.jsx:90:5:React Hook useEffect has a missing dependency: 'setAttachFileSuccess'. Either include it or remove the dependency array.
Linting (Files Changed): src/applications/mhv-secure-messaging/components/AttachmentsList.jsx#L173
src/applications/mhv-secure-messaging/components/AttachmentsList.jsx:173:13:The <va-button> Web Component should be used instead of the button HTML element.
Linting (Files Changed): src/applications/mhv-secure-messaging/components/AttachmentsList.jsx#L272
src/applications/mhv-secure-messaging/components/AttachmentsList.jsx:272:19:The <va-button> Web Component should be used instead of the button HTML element.
Linting (Files Changed): src/applications/mhv-secure-messaging/components/AttachmentsList.jsx#L385
src/applications/mhv-secure-messaging/components/AttachmentsList.jsx:385:3:Prop types declarations should be sorted alphabetically
Linting (Files Changed): src/applications/mhv-secure-messaging/components/shared/AlertBackgroundBox.jsx#L102
src/applications/mhv-secure-messaging/components/shared/AlertBackgroundBox.jsx:102:5:React Hook useEffect has a missing dependency: 'contactListPage'. Either include it or remove the dependency array.

Artifacts

Produced during runtime
Name Size
unit-test-allow-list
34.6 KB