Skip to content

Merge branch 'develop' into feat/CORL-3012-new-commenter-badge #1741

Merge branch 'develop' into feat/CORL-3012-new-commenter-badge

Merge branch 'develop' into feat/CORL-3012-new-commenter-badge #1741

Triggered via push March 15, 2024 13:19
Status Success
Total duration 22m 33s
Artifacts
Fit to window
Zoom out
Zoom in

Annotations

11 warnings
Build and Test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, docker/setup-buildx-action@v2, webfactory/ssh-agent@v0.7.0, docker/login-action@v2, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Build and Test: client/src/core/client/account/routes/email/Confirm/ConfirmForm.tsx#L33
React Hook useCallback has missing dependencies: 'confirm' and 'onSuccess'. Either include them or remove the dependency array. If 'onSuccess' changes too often, find the parent component that defines it and wrap that definition in useCallback
Build and Test: client/src/core/client/account/routes/notifications/Unsubscribe/UnsubscribeForm.tsx#L36
React Hook useCallback has missing dependencies: 'onSuccess' and 'unsubscribe'. Either include them or remove the dependency array. If 'onSuccess' changes too often, find the parent component that defines it and wrap that definition in useCallback
Build and Test: client/src/core/client/account/routes/password/Reset/ResetPasswordForm.tsx#L50
React Hook useCallback has missing dependencies: 'onSuccess' and 'resetPassword'. Either include them or remove the dependency array. If 'onSuccess' changes too often, find the parent component that defines it and wrap that definition in useCallback
Build and Test: client/src/core/client/admin/components/ConversationModal/ConversationModalCommentContainer.tsx#L96
React Hook useCallback has an unnecessary dependency: 'match'. Either exclude it or remove the dependency array
Build and Test: client/src/core/client/admin/components/ConversationModal/ConversationModalHeaderContainer.tsx#L29
React Hook useCallback has missing dependencies: 'comment.story.id' and 'router'. Either include them or remove the dependency array
Build and Test: client/src/core/client/admin/components/StoryInfoDrawer/RecacheStoryAction.tsx#L38
React Hook useCallback has a missing dependency: 'reset'. Either include it or remove the dependency array
Build and Test: client/src/core/client/admin/components/StoryInfoDrawer/RecacheStoryAction.tsx#L50
React Hook useCallback has missing dependencies: 'cached' and 'checkIfCached'. Either include them or remove the dependency array
Build and Test: client/src/core/client/admin/components/UserHistoryDrawer/ModeratorNote.tsx#L28
React Hook useCallback has a missing dependency: 'onDelete'. Either include it or remove the dependency array. If 'onDelete' changes too often, find the parent component that defines it and wrap that definition in useCallback
Build and Test: client/src/core/client/admin/components/UserHistoryDrawer/UserDrawerAccountHistory.tsx#L53
The 'system' JSX element makes the dependencies of useMemo Hook (at line 304) change on every render. Move it inside the useMemo callback. Alternatively, wrap the initialization of 'system' in its own useMemo() Hook
Build and Test: client/src/core/client/admin/components/UserHistoryDrawer/UserDrawerAccountHistory.tsx#L120
React Hook useCallback has an unnecessary dependency: 'getMessage'. Either exclude it or remove the dependency array. Outer scope values like 'getMessage' aren't valid dependencies because mutating them doesn't re-render the component