Add eslint rules #5
pull.yml
on: pull_request
Validate code
/
Unit tests
25s
Validate code
/
Code quality
39s
Annotations
3 errors and 7 warnings
Validate code / Code quality:
src/app.controller.ts#L2
'ApiTags' is defined but never used
|
Validate code / Code quality:
src/app.controller.ts#L7
Controllers should use @ApiTags decorator. This makes it much easier to navigate swagger UI. This ApiTags decorator is in the @nestjs/swagger package on npm
|
Validate code / Code quality
Process completed with exit code 1.
|
Validate code / Code quality:
src/auth/guards/role.guard.ts#L2
Requires 'eslint-enable' directive for '@typescript-eslint/no-unsafe-call'
|
Validate code / Code quality:
src/auth/guards/role.guard.ts#L3
Requires 'eslint-enable' directive for '@typescript-eslint/no-unsafe-member-access'
|
Validate code / Code quality:
src/auth/strategies/admin.strategy.ts#L2
Requires 'eslint-enable' directive for '@typescript-eslint/no-unsafe-call'
|
Validate code / Code quality:
src/auth/strategies/admin.strategy.ts#L3
Requires 'eslint-enable' directive for '@typescript-eslint/no-unsafe-return'
|
Validate code / Code quality:
src/auth/strategies/admin.strategy.ts#L4
Requires 'eslint-enable' directive for '@typescript-eslint/no-explicit-any'
|
Validate code / Code quality:
src/auth/strategies/admin.strategy.ts#L20
Potential timing attack, left side: true
|
Validate code / Code quality:
test/app.e2e-spec.ts#L19
Test has no assertions
|