Continuous Integration

[#IOPID-2557] Minor fixes to previous PR #505

Sign in to view logs

Summary
Jobs
- code_review
- e2e
Run details
- Usage
- Workflow file

Triggered via pull request December 19, 2024 16:52

ready_for_review #151

IOPID-2557-fix

Status Success

Total duration 3m 37s

Artifacts –

code_review.yaml

on: pull_request

Annotations

12 warnings

code_review

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

code_review: apps/io-session-manager/src/__mocks__/appinsights.mocks.ts#L12

Unexpected any. Specify a different type

code_review: apps/io-session-manager/src/controllers/__tests__/authentication.spec.ts#L24

'TelemetryClient' is defined but never used

code_review: apps/io-session-manager/src/controllers/__tests__/pagopa.spec.ts#L176

Update this function so that its implementation is not identical to the one on line 141

code_review: apps/io-session-manager/src/services/__tests__/redis-session-storage.spec.ts#L528

Unexpected let, use const instead

code_review: apps/io-session-manager/src/services/__tests__/redis-session-storage.spec.ts#L538

Unexpected let, use const instead

code_review: apps/io-session-manager/src/services/__tests__/redis-session-storage.spec.ts#L563

Unexpected let, use const instead

code_review: apps/io-session-manager/src/services/__tests__/redis-session-storage.spec.ts#L572

Unexpected let, use const instead

code_review: apps/io-session-manager/src/services/__tests__/redis-session-storage.spec.ts#L584

Unexpected let, use const instead

code_review: apps/io-session-manager/src/utils/__tests__/lollipop.spec.ts#L298

Define a constant instead of duplicating this literal 3 times

code_review: apps/io-session-manager/src/utils/__tests__/redis-trace-wrapper.spec.ts#L1

'beforeAll' is defined but never used

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636