robinp2000/dachshund-auth-provider
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

chore: remove unused imports in the frontend #25

Merged
robinp2000 merged 1 commit from robinp2000/chore/frontend-lint-unused-imports into main 2026-10-18 18:55:55 +02:00 AGit
Conversation 0 Commits 1 Files changed 15 +18 -43
robinp2000 commented 2026-10-18 18:52:51 +02:00
Owner
Copy link

What

Remove 34 imported names in 15 files that ESLint reports as unused
(@typescript-eslint/no-unused-vars). Only import statements change.

Why

Part of bringing npm run lint to zero so that it can be enforced in CI.
A name that is not referenced in the TypeScript is not in the component's
imports array either, so removing its import does not change behavior. The
templates of the affected components were checked for attribute directives
(routerLink, mat-button, mat-icon-button, matInput, mat-dialog-title)
that would silently stop working; there are none.

Tests

  • ng build succeeds.
  • npm run format:check passes.
  • npm run lint findings: 91 -> 57.
  • The frontend has no unit tests.
## What Remove 34 imported names in 15 files that ESLint reports as unused (`@typescript-eslint/no-unused-vars`). Only import statements change. ## Why Part of bringing `npm run lint` to zero so that it can be enforced in CI. A name that is not referenced in the TypeScript is not in the component's `imports` array either, so removing its import does not change behavior. The templates of the affected components were checked for attribute directives (`routerLink`, `mat-button`, `mat-icon-button`, `matInput`, `mat-dialog-title`) that would silently stop working; there are none. ## Tests - `ng build` succeeds. - `npm run format:check` passes. - `npm run lint` findings: 91 -> 57. - The frontend has no unit tests.
chore: remove unused imports in the frontend
All checks were successful
CI / lint (pull_request) Successful in 14s
Details
CI / build (pull_request) Successful in 2m13s
Details
050ec1bad5 
## What
Remove 34 imported names in 15 files that ESLint reports as unused
(`@typescript-eslint/no-unused-vars`). Only import statements change.

## Why
Part of bringing `npm run lint` to zero so that it can be enforced in CI.
A name that is not referenced in the TypeScript is not in the component's
`imports` array either, so removing its import does not change behavior. The
templates of the affected components were checked for attribute directives
(`routerLink`, `mat-button`, `mat-icon-button`, `matInput`, `mat-dialog-title`)
that would silently stop working; there are none.

## Tests
- `ng build` succeeds.
- `npm run format:check` passes.
- `npm run lint` findings: 91 -> 57.
- The frontend has no unit tests.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
robinp2000/dachshund-auth-provider!25
No description provided.