Skip to content

Merge pull request #4536 from coralproject/develop #73

Merge pull request #4536 from coralproject/develop

Merge pull request #4536 from coralproject/develop #73

Triggered via push February 14, 2024 19:07
Status Success
Total duration 36m 11s
Billable time 35m
Artifacts
Build, Test, and Deploy
34m 49s
Build, Test, and Deploy
Fit to window
Zoom out
Zoom in

Annotations

11 warnings
Build, Test, and Deploy
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, google-github-actions/auth@v1, google-github-actions/setup-gcloud@v1, docker/login-action@v2, actions/setup-node@v3, actions/cache@v3, docker/build-push-action@v4. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Build, Test, and Deploy: 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, Test, and Deploy: 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, Test, and Deploy: 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, Test, and Deploy: 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, Test, and Deploy: 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, Test, and Deploy: 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, Test, and Deploy: 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, Test, and Deploy: 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, Test, and Deploy: 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, Test, and Deploy: 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