Skip to content

Update and standardize all sign-in/verify CTA alerts to VADS #65870

Update and standardize all sign-in/verify CTA alerts to VADS

Update and standardize all sign-in/verify CTA alerts to VADS #65870

Triggered via pull request January 17, 2025 17:11
Status Success
Total duration 17m 8s
Artifacts 2

pull-request.yml

on: pull_request
Fetch Unit Test Stability Allow List
47s
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 38s
Linting (Files Changed)
Check Cross App Imports
16m 5s
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/personalization/dashboard/components/Dashboard.jsx#L198
src/applications/personalization/dashboard/components/Dashboard.jsx:198:3:Prop types declarations should be sorted alphabetically
Linting (Files Changed): src/applications/personalization/profile/components/direct-deposit/alerts/VerifyIdentity.jsx#L4
src/applications/personalization/profile/components/direct-deposit/alerts/VerifyIdentity.jsx:4:42:'dataTestId' is missing in props validation
Linting (Files Changed): src/applications/simple-forms/20-10207/containers/IntroductionPage.jsx#L41
src/applications/simple-forms/20-10207/containers/IntroductionPage.jsx:41:6:React Hook useEffect has missing dependencies: 'dispatch' and 'formData'. Either include them or remove the dependency array.
Linting (Files Changed): src/applications/simple-forms/21-0845/containers/IntroductionPage.jsx#L106
src/applications/simple-forms/21-0845/containers/IntroductionPage.jsx:106:15:Missing parentheses around multilines JSX
Linting (Files Changed): src/applications/simple-forms/21-0966/containers/IntroductionPage.jsx#L152
src/applications/simple-forms/21-0966/containers/IntroductionPage.jsx:152:15:Missing parentheses around multilines JSX
Linting (Files Changed): src/applications/simple-forms/21-4142/containers/IntroductionPage.jsx#L16
src/applications/simple-forms/21-4142/containers/IntroductionPage.jsx:16:43:'userIdVerified' is missing in props validation
Linting (Files Changed): src/applications/simple-forms/21-4142/containers/IntroductionPage.jsx#L16
src/applications/simple-forms/21-4142/containers/IntroductionPage.jsx:16:59:'userLoggedIn' is missing in props validation
Linting (Files Changed): src/applications/static-pages/cta-widget/index.js#L45
src/applications/static-pages/cta-widget/index.js:45:3:'propTypes' should be declared outside the class body.
Linting (Files Changed): src/applications/static-pages/cta-widget/index.js#L52
src/applications/static-pages/cta-widget/index.js:52:5:Prop types declarations should be sorted alphabetically
Linting (Files Changed): src/applications/static-pages/cta-widget/index.js#L53
src/applications/static-pages/cta-widget/index.js:53:5:Prop types declarations should be sorted alphabetically
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.29 KB
unit-test-allow-list
32.7 KB