Merge pull request #4536 from coralproject/develop #73
build-test-deploy.yml
on: push
Build, Test, and Deploy
34m 49s
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
|