ci(test): run the test job on windows too - #272
Merged
Merged
Annotations
11 warnings
|
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 npm run lint:ci:
src/renderer/src/features/launch/hooks/useNotifyOnPreventedAppClose.ts#L14
React Hook useEffect has missing dependencies: 'addNotification' and 't'. Either include them or remove the dependency array
|
|
Run npm run lint:ci:
src/renderer/src/features/installations/pages/EditInstallation.tsx#L80
React Hook useEffect has missing dependencies: 'customIcons', 'fields', and 'gameVersions'. Either include them or remove the dependency array
|
|
Run npm run lint:ci:
src/renderer/src/features/installations/pages/EditInstallation.tsx#L65
React Hook useEffect has a missing dependency: 'installations'. Either include it or remove the dependency array
|
|
Run npm run lint:ci:
src/renderer/src/features/config/contexts/ConfigContext.tsx#L120
React Hook useEffect has a missing dependency: 'config.lastUsedInstallation'. Either include it or remove the dependency array
|
|
Run npm run lint:ci:
src/renderer/src/features/config/contexts/ConfigContext.tsx#L108
React Hook useEffect has missing dependencies: 'config.installations' and 'getInstalledMods'. Either include them or remove the dependency array
|
|
Run npm run lint:ci:
src/renderer/src/features/config/contexts/ConfigContext.tsx#L86
React Hook useEffect has missing dependencies: 'addNotification', 'isConfigLoaded', and 't'. Either include them or remove the dependency array
|
|
Run npm run lint:ci:
src/renderer/src/contexts/NotificationsContext.tsx#L120
React Hook useEffect has missing dependencies: 'addNotification' and 't'. Either include them or remove the dependency array
|
|
Run npm run lint:ci:
src/renderer/src/components/ui/StickyMenu.tsx#L37
React Hook useEffect has missing dependencies: 'handleScroll' and 'scrollRef'. Either include them or remove the dependency array
|
|
Run npm run lint:ci:
src/renderer/src/components/ui/StickyMenu.tsx#L35
The ref value 'scrollRef.current' will likely have changed by the time this effect cleanup function runs. If this ref points to a node rendered by React, copy 'scrollRef.current' to a variable inside the effect, and use that variable in the cleanup function
|
|
Run npm run lint:ci:
src/renderer/src/components/layout/GlobalModUpdateChecker.tsx#L37
React Hook useEffect has missing dependencies: 'addNotification', 'configDispatch', 'getCompleteInstalledMods', 'goTo', 'installations', 'notifiedInstallations', and 't'. Either include them or remove the dependency array
|
background
wait
wait-all
cancel
parallel
Loading