Code QA

update legends thumbnails #228

Sign in to view logs

Triggered via push October 9, 2023 18:35

djobbo

master

Status Failure

Total duration 1m 19s

Artifacts –

This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

codeqa-analysis.yml

on: push

Matrix: Analyze

Annotations

10 errors and 4 warnings

Analyze (Lint, pnpm ci:lint): packages/dl-roster-images/index.ts#L33

Unexpected console statement

Analyze (Lint, pnpm ci:lint): packages/dl-roster-images/index.ts#L35

'i' is defined but never used

Analyze (Lint, pnpm ci:lint): packages/server/router/getBHArticles.ts#L3

'parseBHArticlesPage' is defined but never used

Analyze (Lint, pnpm ci:lint): packages/server/router/getBHArticles.ts#L5

'logInfo' is defined but never used

Analyze (Lint, pnpm ci:lint): packages/server/router/getBHArticles.ts#L18

'req' is defined but never used

Analyze (Lint, pnpm ci:lint): packages/server/router/getBHArticles.ts#L19

Delete `⏎`

Analyze (Lint, pnpm ci:lint): packages/server/router/getWeeklyRotation.ts#L1

'getWeeklyRotationFn' is defined but never used

Analyze (Lint, pnpm ci:lint): packages/server/router/stats/getClanStats.ts#L2

'logError' is defined but never used

Analyze (Lint, pnpm ci:lint): packages/server/router/stats/getClanStats.ts#L5

'updateDBClanData' is defined but never used

Analyze (Lint, pnpm ci:lint): packages/server/router/stats/getClanStats.ts#L6

'waitForRequestTimeout' is defined but never used

Analyze (Check Deadcode, pnpm ci:deadcode)

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Analyze (Lint, pnpm ci:lint)

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Analyze (Lint, pnpm ci:lint): packages/common/hooks/useLocalStorageState.ts#L50

React Hook useEffect has a missing dependency: 'initialValue'. Either include it or remove the dependency array. If 'setStoredValue' needs the current value of 'initialValue', you can also switch to useReducer instead of useState and read 'initialValue' in the reducer

Analyze (Check Types, pnpm ts:check)

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-node@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/