qodana improvements #5
JessicaMuleinAnnotations
10 errors, 8 warnings, and 14 notices
|
ESLint:
src/components/screens/DominionAssistantScreen.tsx#L4
ESLint: 'useLocation' is defined but never used. (@typescript-eslint/no-unused-vars)
|
|
ESLint:
src/game/dominion-lib.ts#L239
ESLint: Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
|
|
ESLint:
src/game/utils.ts#L47
ESLint: Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
|
|
ESLint:
src/game/__tests__/dominion-lib-fieldSubfieldToGameLogAction.spec.ts#L164
ESLint: Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
|
|
ESLint:
src/components/screens/DominionAssistantScreen.tsx#L4
ESLint: 'useNavigate' is defined but never used. (@typescript-eslint/no-unused-vars)
|
|
ESLint:
src/game/utils.ts#L38
ESLint: Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
|
|
ESLint:
src/game/utils.ts#L24
ESLint: Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
|
|
ESLint:
src/game/dominion-lib.ts#L256
ESLint: Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
|
|
ESLint:
src/game/__tests__/dominion-lib-fieldSubfieldToGameLogAction.spec.ts#L170
ESLint: Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
|
|
ESLint:
src/game/dominion-lib.ts#L257
ESLint: Unexpected any. Specify a different type. (@typescript-eslint/no-explicit-any)
|
|
Unused import:
src/components/screens/DominionAssistantScreen.tsx#L4
Unused
```
import { useNavigate, useLocation } from 'react-router-dom';
```
|
|
Exception used for local control-flow:
src/game/dominion-lib-log.ts#L784
'throw' of exception caught locally
|
|
Exception used for local control-flow:
src/game/dominion-lib-log.ts#L829
'throw' of exception caught locally
|
|
Exception used for local control-flow:
src/game/dominion-lib-log.ts#L795
'throw' of exception caught locally
|
|
Exception used for local control-flow:
src/game/dominion-lib-load-save.ts#L266
'throw' of exception caught locally
|
|
Exception used for local control-flow:
src/game/dominion-lib-load-save.ts#L214
'throw' of exception caught locally
|
|
Redundant local variable:
src/game/dominion-lib.ts#L133
Local variable newPlayer is redundant
|
|
qodana
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Deprecated symbol used:
src/game/disk-storage-service.ts#L38
Deprecated symbol used, consult docs for better alternative
|
|
Result of method call returning a promise is ignored:
src/components/screens/AboutScreen.tsx#L30
Promise returned from fetchMessages is ignored
|