Skip to content

UI Checkstyle

UI Checkstyle #29428

Triggered via merge group September 9, 2026 13:08
Status Success
Total duration 6m 53s
Artifacts

ui-checkstyle.yml

on: merge_group
Fit to window
Zoom out
Zoom in

Annotations

18 warnings
checkstyle: openmetadata-ui/src/main/resources/ui/playwright/utils/user.ts#L610
Use waitForAggregation from playwright/utils/searchAggregation instead of waiting on search/aggregate directly — a wait that names only the endpoint or field also matches the dropdown-open request and can resolve early
checkstyle: openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/Teams.spec.ts#L154
Prefer the `page` fixture (test.use({ storageState })) over browser.newPage() + manual login for single-user admin tests. For multi-user tests that need a second non-admin page, this warning is expected — no action needed
checkstyle: openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/Teams.spec.ts#L148
Prefer the `page` fixture (test.use({ storageState })) over browser.newPage() + manual login for single-user admin tests. For multi-user tests that need a second non-admin page, this warning is expected — no action needed
checkstyle: openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/Teams.spec.ts#L142
Prefer the `page` fixture (test.use({ storageState })) over browser.newPage() + manual login for single-user admin tests. For multi-user tests that need a second non-admin page, this warning is expected — no action needed
checkstyle: openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/Teams.spec.ts#L136
Prefer the `page` fixture (test.use({ storageState })) over browser.newPage() + manual login for single-user admin tests. For multi-user tests that need a second non-admin page, this warning is expected — no action needed
checkstyle: openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DataModel/DataModels/ModelTab/ModelTab.component.tsx#L191
React Hook useMemo has a missing dependency: 'deleted'. Either include it or remove the dependency array
checkstyle: openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DataModel/DataModels/ModelTab/ModelTab.component.tsx#L167
React Hook useCallback has an unnecessary dependency: 'paging'. Either exclude it or remove the dependency array
checkstyle: openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DashboardChartTable/DashboardChartTable.tsx#L407
React Hook useMemo has missing dependencies: 'dashboardDetails?.fullyQualifiedName', 'hasEditGlossaryTermAccess', 't', 'tagFilter.Classification', and 'tagFilter.Glossary'. Either include them or remove the dependency array
checkstyle: openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DashboardChartTable/DashboardChartTable.tsx#L291
React Hook useCallback has an unnecessary dependency: 'chartFilters'. Either exclude it or remove the dependency array
checkstyle: openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DashboardChartTable/DashboardChartTable.tsx#L156
React Hook useCallback has a missing dependency: 't'. Either include it or remove the dependency array
checkstyle: openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DashboardChartTable/DashboardChartTable.tsx#L136
React Hook useEffect has a missing dependency: 'getAllChartsPermissions'. Either include it or remove the dependency array
checkstyle: openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DashboardChartTable/DashboardChartTable.tsx#L129
React Hook useCallback has missing dependencies: 'fetchChartPermissions' and 't'. Either include them or remove the dependency array
checkstyle: openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DashboardChartTable/DashboardChartTable.tsx#L104
React Hook useCallback has a missing dependency: 'getEntityPermission'. Either include it or remove the dependency array
checkstyle: openmetadata-ui/src/main/resources/ui/src/components/Container/ContainerDataModel/ContainerDataModel.tsx#L308
React Hook useMemo has missing dependencies: 't', 'tagFilter.Classification', and 'tagFilter.Glossary'. Either include them or remove the dependency array. Outer scope values like 'getEntityName' aren't valid dependencies because mutating them doesn't re-render the component
checkstyle: openmetadata-ui/src/main/resources/ui/src/components/APIEndpoint/APIEndpointSchema/APIEndpointSchema.tsx#L471
React Hook useMemo has a missing dependency: 't'. Either include it or remove the dependency array