Skip to content
Node.js CI

feat(lint): making unused vars an error and fixing lint #35815

Sign in to view logs

feat(lint): making unused vars an error and fixing lint

feat(lint): making unused vars an error and fixing lint #35815

Triggered via pull request June 17, 2026 08:10
@kdinevkdinev
synchronize #17316
kdinev/unused-vars-lint
Status Failure
Total duration 7m 32s
Artifacts –

nodejs.yml

on: pull_request
run-tests
7m 23s
run-tests
Fit to window
Zoom out
Zoom in

Annotations

1 error and 12 warnings
run-tests
Process completed with exit code 1.
run-tests: projects/igniteui-angular-extras/src/lib/context-menu/context-menu.component.ts#L53
Components should not opt out of the default `ChangeDetectionStrategy.OnPush` change detection strategy
run-tests: projects/igniteui-angular-extras/src/lib/context-menu/chart-dialog/chart-dialog.component.ts#L29
Components should not opt out of the default `ChangeDetectionStrategy.OnPush` change detection strategy
run-tests: src/app/avatar/avatar.sample.ts#L31
Components should not opt out of the default `ChangeDetectionStrategy.OnPush` change detection strategy
run-tests: src/app/autocomplete/autocomplete.sample.ts#L56
Prefer using the inject() function over constructor parameter injection. Use Angular's migration schematic to automatically refactor: ng generate @angular/core:inject
run-tests: src/app/autocomplete/autocomplete.sample.ts#L28
Components should not opt out of the default `ChangeDetectionStrategy.OnPush` change detection strategy
run-tests: src/app/app.component.ts#L828
Prefer using the inject() function over constructor parameter injection. Use Angular's migration schematic to automatically refactor: ng generate @angular/core:inject
run-tests: src/app/app.component.ts#L828
Prefer using the inject() function over constructor parameter injection. Use Angular's migration schematic to automatically refactor: ng generate @angular/core:inject
run-tests: src/app/app.component.ts#L41
Components should not opt out of the default `ChangeDetectionStrategy.OnPush` change detection strategy
run-tests: src/app/action-strip/action-strip.sample.ts#L64
Components should not opt out of the default `ChangeDetectionStrategy.OnPush` change detection strategy
run-tests: src/app/accordion/accordion.sample.ts#L40
Prefer using the inject() function over constructor parameter injection. Use Angular's migration schematic to automatically refactor: ng generate @angular/core:inject
run-tests: src/app/accordion/accordion.sample.ts#L39
Prefer using the inject() function over constructor parameter injection. Use Angular's migration schematic to automatically refactor: ng generate @angular/core:inject
run-tests: src/app/accordion/accordion.sample.ts#L22
Components should not opt out of the default `ChangeDetectionStrategy.OnPush` change detection strategy