Skip to content

Final end to end test and fixed #637

Final end to end test and fixed

Final end to end test and fixed #637

Triggered via pull request June 2, 2024 17:26
Status Success
Total duration 48s
Artifacts

lint.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
linting: components/Graph/ForceDirectedGraph/ForceDirectedGraph.tsx#L98
React Hook useEffect has missing dependencies: 'graphState.links', 'graphState.nodes', 'height', 'simulationParameters?.center', 'simulationParameters?.charge', 'simulationParameters?.collide', 'simulationParameters?.distance', and 'width'. Either include them or remove the dependency array. You can also replace multiple useState variables with useReducer if 'setGraphState' needs the current value of 'graphState.nodes'
linting: components/Graph/ForceDirectedGraph/ForceDirectedGraph.tsx#L113
React Hook useEffect has missing dependencies: 'height' and 'width'. Either include them or remove the dependency array
linting: components/Graph/ForceDirectedGraph/ForceDirectedGraph.tsx#L190
React Hook useCallback has a missing dependency: 'transformCoordinates'. Either include it or remove the dependency array
linting: components/Graph/GraphOptionsModal/GraphOptionsModal.tsx#L30
React Hook useEffect has a missing dependency: 'updateSimulationParameters'. Either include it or remove the dependency array. If 'updateSimulationParameters' changes too often, find the parent component that defines it and wrap that definition in useCallback
linting: components/Graph/NodeModal/NodeModal.tsx#L59
React Hook useEffect has a missing dependency: 'slideInModal'. Either include it or remove the dependency array
linting: screens/Explore/AnnouncementScreen/AnnouncementScreen.tsx#L66
React Hook useEffect has a missing dependency: 'recommendAnnouncements'. Either include it or remove the dependency array
linting: screens/Explore/AnnouncementScreen/AnnouncementScreen.tsx#L86
React Hook useEffect has missing dependencies: 'announcements', 'recommendAnnouncements', and 'user_id'. Either include them or remove the dependency array. You can also replace multiple useState variables with useReducer if 'setFilteredAnnouncements' needs the current value of 'announcements'
linting: screens/Explore/AnnouncementScreen/AnnouncementScreen.tsx#L101
React Hook useEffect has a missing dependency: 'recommendAnnouncements'. Either include it or remove the dependency array
linting: screens/Network/ContactGraph/ContactGraph.tsx#L81
React Hook useEffect has a missing dependency: 'fullScreenCallback'. Either include it or remove the dependency array. If 'fullScreenCallback' changes too often, find the parent component that defines it and wrap that definition in useCallback
linting: screens/Network/ContactGraph/ContactGraph.tsx#L262
React Hook useEffect has missing dependencies: 'changeTab', 'graph', 'magicNeighbors', and 'userContact.friends'. Either include them or remove the dependency array. If 'changeTab' changes too often, find the parent component that defines it and wrap that definition in useCallback