Refactor chat shortcuts and notifications, fix linting issues - #1830
Open
TanCodeX wants to merge 5 commits into
Open
Refactor chat shortcuts and notifications, fix linting issues#1830TanCodeX wants to merge 5 commits into
TanCodeX wants to merge 5 commits into
Annotations
2 errors and 3 warnings
|
Run Tests with Coverage
Process completed with exit code 1.
|
|
Run Tests with Coverage:
src/hooks/useSkillEndorsements.ts#L49
TypeError: supabase.auth.onAuthStateChange is not a function
❯ src/hooks/useSkillEndorsements.ts:49:54
❯ commitHookEffectListMount node_modules/react-dom/cjs/react-dom.development.js:23189:26
❯ commitPassiveMountOnFiber node_modules/react-dom/cjs/react-dom.development.js:24970:11
❯ commitPassiveMountEffects_complete node_modules/react-dom/cjs/react-dom.development.js:24930:9
❯ commitPassiveMountEffects_begin node_modules/react-dom/cjs/react-dom.development.js:24917:7
❯ commitPassiveMountEffects node_modules/react-dom/cjs/react-dom.development.js:24905:3
❯ flushPassiveEffectsImpl node_modules/react-dom/cjs/react-dom.development.js:27078:3
❯ flushPassiveEffects node_modules/react-dom/cjs/react-dom.development.js:27023:14
❯ node_modules/react-dom/cjs/react-dom.development.js:26808:9
❯ flushActQueue node_modules/react/cjs/react.development.js:2667:24
|
|
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/
|
|
Run Linter:
src/hooks/useSessions.ts#L346
React Hook useCallback has an unnecessary dependency: 'toast'. Either exclude it or remove the dependency array. Outer scope values like 'toast' aren't valid dependencies because mutating them doesn't re-render the component
|
|
Run Linter:
src/hooks/useSessions.ts#L199
React Hook useEffect has an unnecessary dependency: 'toast'. Either exclude it or remove the dependency array. Outer scope values like 'toast' aren't valid dependencies because mutating them doesn't re-render the component
|
background
wait
wait-all
cancel
parallel
Loading