Skip to content
frontend PR

Dynamic notices fix for table detail page #1417

Sign in to view logs

Dynamic notices fix for table detail page

Dynamic notices fix for table detail page #1417

Triggered via pull request November 14, 2023 06:07
@mikaalanwarmikaalanwar
opened #2210
deliveryhero:dynamic-notices-fix
Status Failure
Total duration 6m 53s
Artifacts –

frontend_pull_request.yml

on: pull_request
Matrix: test-unit-frontend
Matrix: test-unit-python
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 13 warnings
test-unit-frontend (12.x): frontend/amundsen_application/static/js/pages/TableDetailPage/index.tsx#L757
Replace `<AlertList·notices={aggregatedTableNotices}·/>` with `(⏎················<AlertList·notices={aggregatedTableNotices}·/>⏎··············)`
test-unit-frontend (12.x)
Process completed with exit code 1.
test-unit-python (3.8.x)
Process completed with exit code 2.
test-unit-python (3.7.x)
The operation was canceled.
test-unit-frontend (12.x)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
test-unit-frontend (12.x): frontend/amundsen_application/static/js/components/Bookmark/BookmarkIcon/index.tsx#L72
Visible, non-interactive elements with click handlers must have at least one keyboard listener
test-unit-frontend (12.x): frontend/amundsen_application/static/js/components/Bookmark/BookmarkIcon/index.tsx#L72
Static HTML elements with event handlers require a role
test-unit-frontend (12.x): frontend/amundsen_application/static/js/components/EditableSection/index.tsx#L143
A form label must be associated with a control
test-unit-frontend (12.x): frontend/amundsen_application/static/js/components/EditableText/index.tsx#L84
Do not use setState in componentDidUpdate
test-unit-frontend (12.x): frontend/amundsen_application/static/js/components/EditableText/index.tsx#L102
Do not use setState in componentDidUpdate
test-unit-frontend (12.x): frontend/amundsen_application/static/js/components/EntityCard/EntityCardSection/index.tsx#L68
A control must be associated with a text label
test-unit-frontend (12.x): frontend/amundsen_application/static/js/components/FlashMessage/flashMessage.story.tsx#L19
Unexpected alert
test-unit-frontend (12.x): frontend/amundsen_application/static/js/components/FlashMessage/flashMessage.story.tsx#L28
Unexpected alert
test-unit-frontend (12.x): frontend/amundsen_application/static/js/components/Inputs/CheckBoxItem/index.spec.tsx#L36
Use object destructuring
test-unit-frontend (12.x): frontend/amundsen_application/static/js/components/Inputs/CheckBoxItem/index.spec.tsx#L37
Use object destructuring
test-unit-python (3.8.x)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-python@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
test-unit-python (3.7.x)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-python@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/