feat: implement comprehensive logging standards (#536)#962
Open
soterika wants to merge 2 commits into
Open
Annotations
10 errors and 2 warnings
|
Run lint:
src/logging/redaction.util.spec.ts#L54
Delete `····`
|
|
Run lint:
src/logging/logging.interceptor.ts#L9
'rxjs' import is duplicated
|
|
Run lint:
src/logging/logging.interceptor.ts#L1
Replace `⏎··CallHandler,⏎··ExecutionContext,⏎··Injectable,⏎··NestInterceptor,⏎` with `·CallHandler,·ExecutionContext,·Injectable,·NestInterceptor·`
|
|
Run lint:
src/logging/logging.interceptor.spec.ts#L9
Replace `⏎··method·=·'GET',⏎··url·=·'/test',⏎··statusCode·=·200,⏎` with `method·=·'GET',·url·=·'/test',·statusCode·=·200`
|
|
Run lint:
src/logging/app-logger.service.spec.ts#L35
Replace `.spyOn(Object.getPrototypeOf(Object.getPrototypeOf(logger))·as·ConsoleLogger,·'fatal')` with `⏎······.spyOn(Object.getPrototypeOf(Object.getPrototypeOf(logger))·as·ConsoleLogger,·'fatal')⏎······`
|
|
Run lint:
src/logging/app-logger.service.spec.ts#L34
Replace `··=·jest.spyOn(Object.getPrototypeOf(Object.getPrototypeOf(logger))·as·ConsoleLogger,·'error')` with `=·jest⏎······.spyOn(Object.getPrototypeOf(Object.getPrototypeOf(logger))·as·ConsoleLogger,·'error')⏎······`
|
|
Run lint:
src/logging/app-logger.service.spec.ts#L33
Replace `·=·jest.spyOn(Object.getPrototypeOf(Object.getPrototypeOf(logger))·as·ConsoleLogger,·'warn')` with `=·jest⏎······.spyOn(Object.getPrototypeOf(Object.getPrototypeOf(logger))·as·ConsoleLogger,·'warn')⏎······`
|
|
Run lint:
src/logging/app-logger.service.spec.ts#L32
Replace `··=·jest.spyOn(Object.getPrototypeOf(Object.getPrototypeOf(logger))·as·ConsoleLogger,·'log')` with `=·jest⏎······.spyOn(Object.getPrototypeOf(Object.getPrototypeOf(logger))·as·ConsoleLogger,·'log')⏎······`
|
|
Run lint:
src/logging/app-logger.service.spec.ts#L31
Replace `.spyOn(Object.getPrototypeOf(Object.getPrototypeOf(logger))·as·ConsoleLogger,·'debug')` with `⏎······.spyOn(Object.getPrototypeOf(Object.getPrototypeOf(logger))·as·ConsoleLogger,·'debug')⏎······`
|
|
Run lint:
src/logging/app-logger.service.spec.ts#L27
Replace `·records.push(JSON.parse(String(serialised))·as·IAppLogRecord);·}·catch·{·/*·ignore·*/` with `⏎········records.push(JSON.parse(String(serialised))·as·IAppLogRecord);⏎······}·catch·{⏎········/*·ignore·*/⏎·····`
|
|
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, pnpm/action-setup@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Run lint:
src/auth/auth.service.ts#L9
'isRS256Configured' is defined but never used. Allowed unused vars must match /^_/u
|
background
wait
wait-all
cancel
parallel
Loading