Fix UploadPage smoke test assertion after upload page changes#65
Merged
Annotations
10 errors and 8 warnings
|
Lint:
PanTS-Demo/src/helpers/CornerstoneNifti2.tsx#L148
Unexpected any. Specify a different type
|
|
Lint:
PanTS-Demo/src/helpers/CornerstoneNifti2.tsx#L148
Fast refresh only works when a file only exports components. Use a new file to share constants or functions between components
|
|
Lint:
PanTS-Demo/src/helpers/CornerstoneNifti2.tsx#L132
'_crosshairChangeCallbacks' is never reassigned. Use 'const' instead
|
|
Lint:
PanTS-Demo/src/contexts/fileContexts.tsx#L9
Fast refresh only works when a file only exports components. Move your React context(s) to a separate file
|
|
Lint:
PanTS-Demo/src/contexts/annotationContexts.tsx#L13
Fast refresh only works when a file only exports components. Move your React context(s) to a separate file
|
|
Lint:
PanTS-Demo/src/components/ReportScreen/ReportScreen.tsx#L433
Empty block statement
|
|
Lint:
PanTS-Demo/src/components/ReportScreen/ReportScreen.tsx#L122
Unnecessary escape character: \-
|
|
Lint:
PanTS-Demo/src/components/OrganMesh.tsx#L12
Fast refresh only works when a file only exports components. Use a new file to share constants or functions between components
|
|
Lint:
PanTS-Demo/src/components/MeshViewer.tsx#L19
Fast refresh only works when a file only exports components. Use a new file to share constants or functions between components
|
|
Lint:
PanTS-Demo/src/components/InferenceUpload.tsx#L10
Unexpected any. Specify a different type
|
|
Complete job
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-node@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Lint:
PanTS-Demo/src/routes/VisualizationPage.tsx#L1062
The 'handleWindowChange' function makes the dependencies of useMemo Hook (at line 1562) change on every render. To fix this, wrap the definition of 'handleWindowChange' in its own useCallback() Hook
|
|
Lint:
PanTS-Demo/src/components/ReportScreen/ReportScreen.tsx#L461
React Hook useEffect has missing dependencies: 'curOrganData', 'curOrganName', 'flagged', 'onClearHighlight', 'onHideOrgans', and 'onOrganHighlight'. Either include them or remove the dependency array. If 'onClearHighlight' changes too often, find the parent component that defines it and wrap that definition in useCallback
|
|
Lint:
PanTS-Demo/src/components/ReportScreen/ReportScreen.tsx#L436
React Hook useEffect has a missing dependency: 'fetchPlain'. Either include it or remove the dependency array
|
|
Lint:
PanTS-Demo/src/components/ReportScreen/FindingsTimeline.tsx#L64
React Hook useEffect has a missing dependency: 'ordered'. Either include it or remove the dependency array
|
|
Lint:
PanTS-Demo/src/components/NestedCheckBox/NestedCheckBox.tsx#L92
React Hook useEffect has missing dependencies: 'cacheKey' and 'clabelId'. Either include them or remove the dependency array
|
|
Lint:
PanTS-Demo/src/components/CtPreview/DicomPreview.tsx#L101
React Hook useEffect has a missing dependency: 'slice.total'. Either include it or remove the dependency array
|
|
Lint:
PanTS-Demo/src/components/CtPreview/DicomPreview.tsx#L92
The ref value 'elementRef.current' will likely have changed by the time this effect cleanup function runs. If this ref points to a node rendered by React, copy 'elementRef.current' to a variable inside the effect, and use that variable in the cleanup function
|
background
wait
wait-all
cancel
parallel
Loading