[57405] Use Primer's PageHeader component on the WorkPackage list view #25228
reviewdog [eslint] report
reported by reviewdog 🐶
Findings (29)
frontend/src/app/core/loading-indicator/loading-indicator.service.ts|29 col 1| File has too many classes (2). Maximum allowed is 1.
frontend/src/app/features/work-packages/components/edit-actions-bar/wp-edit-actions-bar.component.ts|46 col 11| Output bindings should not be aliased (https://angular.io/guide/styleguide#style-05-13)
frontend/src/app/features/work-packages/components/edit-actions-bar/wp-edit-actions-bar.component.ts|46 col 11| Output bindings, including aliases, should not be named "on", nor prefixed with it (https://angular.io/guide/styleguide#style-05-16)
frontend/src/app/features/work-packages/components/edit-actions-bar/wp-edit-actions-bar.component.ts|46 col 28| Output bindings, including aliases, should not be named "on", nor prefixed with it (https://angular.io/guide/styleguide#style-05-16)
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|65 col 3| Unsafe assignment of an any value.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|65 col 59| Unsafe member access .baseRoute on an any value.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|76 col 31| Unexpected any. Specify a different type.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|124 col 35| Unsafe member access .projectPath on an any value.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|124 col 91| Unsafe argument of type any assigned to a parameter of type string.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|124 col 108| Unsafe member access .projectPath on an any value.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|197 col 7| Promises must be awaited, end with a call to .catch, end with a call to .then with a rejection handler or be explicitly marked as ignored with the void operator.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|199 col 37| Unsafe member access .projectPath on an any value.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|199 col 91| Unsafe argument of type any assigned to a parameter of type string.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|199 col 108| Unsafe member access .projectPath on an any value.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|206 col 59| Unsafe argument of type any assigned to a parameter of type string.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|206 col 70| Unsafe member access .identifier on an any value.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|213 col 69| Unsafe argument of type any assigned to a parameter of type string.
frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts|213 col 80| Unsafe member access .identifier on an any value.
frontend/src/app/features/work-packages/components/wp-new/wp-new-full-view.component.ts|39 col 54| Unsafe member access .successState on an any value.
frontend/src/app/features/work-packages/components/wp-tabs/components/wp-tab-wrapper/wp-tab-wrapper.component.ts|62 col 50| Missing trailing comma.
frontend/src/app/features/work-packages/routing/wp-list-view/wp-primerized-list-view.component.ts|53 col 3| Use @HostBinding or @HostListener rather than the host metadata property (https://angular.io/styleguide#style-06-03)
frontend/src/app/features/work-packages/routing/wp-list-view/wp-primerized-list-view.component.ts|85 col 9| 'link' is never reassigned. Use 'const' instead.
frontend/src/app/features/work-packages/routing/wp-list-view/wp-primerized-list-view.component.ts|125 col 4| Block must not be padded by blank lines.
frontend/src/app/features/work-packages/services/work-package-authorization.service.ts|85 col 48| Unsafe argument of type any assigned to a parameter of type string | null.
frontend/src/app/shared/components/op-context-menu/handlers/op-types-context-menu.directive.ts|120 col 11| Promises must be awaited, end with a call to .catch, end with a call to .then with a rejection handler or be explicitly marked as ignored with the void operator.
frontend/src/app/shared/components/op-context-menu/wp-context-menu/wp-single-context-menu.ts|66 col 73| Unsafe argument of type any assigned to a parameter of type string | null.
frontend/src/app/shared/components/op-context-menu/wp-context-menu/wp-single-context-menu.ts|66 col 98| Unsafe member access .identifier on an any value.
frontend/src/app/shared/components/op-context-menu/wp-context-menu/wp-view-context-menu.directive.ts|156 col 66| Unsafe argument of type any assigned to a parameter of type string | null.
frontend/src/app/shared/components/op-context-menu/wp-context-menu/wp-view-context-menu.directive.ts|156 col 86| Unsafe member access .id on an any value.
Filtered Findings (0)
Annotations
Check failure on line 140 in frontend/src/app/core/loading-indicator/loading-indicator.service.ts
github-actions / eslint
[eslint] frontend/src/app/core/loading-indicator/loading-indicator.service.ts#L29-L140 <max-classes-per-file>(https://eslint.org/docs/latest/rules/max-classes-per-file)
File has too many classes (2). Maximum allowed is 1.
Raw output
{"ruleId":"max-classes-per-file","severity":2,"message":"File has too many classes (2). Maximum allowed is 1.","line":29,"column":1,"nodeType":"Program","messageId":"maximumExceeded","endLine":140,"endColumn":1}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/edit-actions-bar/wp-edit-actions-bar.component.ts#L46 <@angular-eslint/no-output-rename>(https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/no-output-rename.md)
Output bindings should not be aliased (https://angular.io/guide/styleguide#style-05-13)
Raw output
{"ruleId":"@angular-eslint/no-output-rename","severity":2,"message":"Output bindings should not be aliased (https://angular.io/guide/styleguide#style-05-13)","line":46,"column":11,"nodeType":"Literal","messageId":"noOutputRename","endLine":46,"endColumn":19,"fix":{"range":[1822,1830],"text":""}}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/edit-actions-bar/wp-edit-actions-bar.component.ts#L46 <@angular-eslint/no-output-on-prefix>(https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/no-output-on-prefix.md)
Output bindings, including aliases, should not be named "on", nor prefixed with it (https://angular.io/guide/styleguide#style-05-16)
Raw output
{"ruleId":"@angular-eslint/no-output-on-prefix","severity":2,"message":"Output bindings, including aliases, should not be named \"on\", nor prefixed with it (https://angular.io/guide/styleguide#style-05-16)","line":46,"column":11,"nodeType":"Literal","messageId":"noOutputOnPrefix","endLine":46,"endColumn":19}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/edit-actions-bar/wp-edit-actions-bar.component.ts#L46 <@angular-eslint/no-output-on-prefix>(https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/no-output-on-prefix.md)
Output bindings, including aliases, should not be named "on", nor prefixed with it (https://angular.io/guide/styleguide#style-05-16)
Raw output
{"ruleId":"@angular-eslint/no-output-on-prefix","severity":2,"message":"Output bindings, including aliases, should not be named \"on\", nor prefixed with it (https://angular.io/guide/styleguide#style-05-16)","line":46,"column":28,"nodeType":"Identifier","messageId":"noOutputOnPrefix","endLine":46,"endColumn":34}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L65 <@typescript-eslint/no-unsafe-assignment>(https://typescript-eslint.io/rules/no-unsafe-assignment)
Unsafe assignment of an `any` value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-assignment","severity":2,"message":"Unsafe assignment of an `any` value.","line":65,"column":3,"nodeType":"PropertyDefinition","messageId":"anyAssignment","endLine":65,"endColumn":69}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L65 <@typescript-eslint/no-unsafe-member-access>(https://typescript-eslint.io/rules/no-unsafe-member-access)
Unsafe member access .baseRoute on an `any` value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-member-access","severity":2,"message":"Unsafe member access .baseRoute on an `any` value.","line":65,"column":59,"nodeType":"Identifier","messageId":"unsafeMemberExpression","endLine":65,"endColumn":68}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L76 <@typescript-eslint/no-explicit-any>(https://typescript-eslint.io/rules/no-explicit-any)
Unexpected any. Specify a different type.
Raw output
{"ruleId":"@typescript-eslint/no-explicit-any","severity":2,"message":"Unexpected any. Specify a different type.","line":76,"column":31,"nodeType":"TSAnyKeyword","messageId":"unexpectedAny","endLine":76,"endColumn":34,"suggestions":[{"messageId":"suggestUnknown","fix":{"range":[3498,3501],"text":"unknown"},"desc":"Use `unknown` instead, this will force you to explicitly, and safely assert the type is correct."},{"messageId":"suggestNever","fix":{"range":[3498,3501],"text":"never"},"desc":"Use `never` instead, this is useful when instantiating generic type parameters that you don't need to know the type of."}]}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L124 <@typescript-eslint/no-unsafe-member-access>(https://typescript-eslint.io/rules/no-unsafe-member-access)
Unsafe member access .projectPath on an `any` value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-member-access","severity":2,"message":"Unsafe member access .projectPath on an `any` value.","line":124,"column":35,"nodeType":"Identifier","messageId":"unsafeMemberExpression","endLine":124,"endColumn":46}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L124 <@typescript-eslint/no-unsafe-argument>(https://typescript-eslint.io/rules/no-unsafe-argument)
Unsafe argument of type `any` assigned to a parameter of type `string`.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-argument","severity":2,"message":"Unsafe argument of type `any` assigned to a parameter of type `string`.","line":124,"column":91,"nodeType":"MemberExpression","messageId":"unsafeArgument","endLine":124,"endColumn":119}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L124 <@typescript-eslint/no-unsafe-member-access>(https://typescript-eslint.io/rules/no-unsafe-member-access)
Unsafe member access .projectPath on an `any` value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-member-access","severity":2,"message":"Unsafe member access .projectPath on an `any` value.","line":124,"column":108,"nodeType":"Identifier","messageId":"unsafeMemberExpression","endLine":124,"endColumn":119}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L197 <@typescript-eslint/no-floating-promises>(https://typescript-eslint.io/rules/no-floating-promises)
Promises must be awaited, end with a call to .catch, end with a call to .then with a rejection handler or be explicitly marked as ignored with the `void` operator.
Raw output
{"ruleId":"@typescript-eslint/no-floating-promises","severity":2,"message":"Promises must be awaited, end with a call to .catch, end with a call to .then with a rejection handler or be explicitly marked as ignored with the `void` operator.","line":197,"column":7,"nodeType":"ExpressionStatement","messageId":"floatingVoid","endLine":197,"endColumn":60,"suggestions":[{"messageId":"floatingFixVoid","fix":{"range":[7453,7453],"text":"void "},"desc":"Add void operator to ignore."}]}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L199 <@typescript-eslint/no-unsafe-member-access>(https://typescript-eslint.io/rules/no-unsafe-member-access)
Unsafe member access .projectPath on an `any` value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-member-access","severity":2,"message":"Unsafe member access .projectPath on an `any` value.","line":199,"column":37,"nodeType":"Identifier","messageId":"unsafeMemberExpression","endLine":199,"endColumn":48}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L199 <@typescript-eslint/no-unsafe-argument>(https://typescript-eslint.io/rules/no-unsafe-argument)
Unsafe argument of type `any` assigned to a parameter of type `string`.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-argument","severity":2,"message":"Unsafe argument of type `any` assigned to a parameter of type `string`.","line":199,"column":91,"nodeType":"MemberExpression","messageId":"unsafeArgument","endLine":199,"endColumn":119}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L199 <@typescript-eslint/no-unsafe-member-access>(https://typescript-eslint.io/rules/no-unsafe-member-access)
Unsafe member access .projectPath on an `any` value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-member-access","severity":2,"message":"Unsafe member access .projectPath on an `any` value.","line":199,"column":108,"nodeType":"Identifier","messageId":"unsafeMemberExpression","endLine":199,"endColumn":119}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L206 <@typescript-eslint/no-unsafe-argument>(https://typescript-eslint.io/rules/no-unsafe-argument)
Unsafe argument of type `any` assigned to a parameter of type `string`.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-argument","severity":2,"message":"Unsafe argument of type `any` assigned to a parameter of type `string`.","line":206,"column":59,"nodeType":"MemberExpression","messageId":"unsafeArgument","endLine":206,"endColumn":80}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L206 <@typescript-eslint/no-unsafe-member-access>(https://typescript-eslint.io/rules/no-unsafe-member-access)
Unsafe member access .identifier on an `any` value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-member-access","severity":2,"message":"Unsafe member access .identifier on an `any` value.","line":206,"column":70,"nodeType":"Identifier","messageId":"unsafeMemberExpression","endLine":206,"endColumn":80}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L213 <@typescript-eslint/no-unsafe-argument>(https://typescript-eslint.io/rules/no-unsafe-argument)
Unsafe argument of type `any` assigned to a parameter of type `string`.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-argument","severity":2,"message":"Unsafe argument of type `any` assigned to a parameter of type `string`.","line":213,"column":69,"nodeType":"MemberExpression","messageId":"unsafeArgument","endLine":213,"endColumn":90}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-create.component.ts#L213 <@typescript-eslint/no-unsafe-member-access>(https://typescript-eslint.io/rules/no-unsafe-member-access)
Unsafe member access .identifier on an `any` value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-member-access","severity":2,"message":"Unsafe member access .identifier on an `any` value.","line":213,"column":80,"nodeType":"Identifier","messageId":"unsafeMemberExpression","endLine":213,"endColumn":90}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-new/wp-new-full-view.component.ts#L39 <@typescript-eslint/no-unsafe-member-access>(https://typescript-eslint.io/rules/no-unsafe-member-access)
Unsafe member access .successState on an `any` value.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-member-access","severity":2,"message":"Unsafe member access .successState on an `any` value.","line":39,"column":54,"nodeType":"Identifier","messageId":"unsafeMemberExpression","endLine":39,"endColumn":66}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/components/wp-tabs/components/wp-tab-wrapper/wp-tab-wrapper.component.ts#L62-L63 <@typescript-eslint/comma-dangle>(https://typescript-eslint.io/rules/comma-dangle)
Missing trailing comma.
Raw output
{"ruleId":"@typescript-eslint/comma-dangle","severity":2,"message":"Missing trailing comma.","line":62,"column":50,"nodeType":"TSParameterProperty","messageId":"missing","endLine":63,"endColumn":1,"fix":{"range":[2412,2438],"text":"WorkPackageTabsService,\n )"}}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/routing/wp-list-view/wp-primerized-list-view.component.ts#L53 <@angular-eslint/no-host-metadata-property>(https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/no-host-metadata-property.md)
Use @HostBinding or @HostListener rather than the `host` metadata property (https://angular.io/styleguide#style-06-03)
Raw output
{"ruleId":"@angular-eslint/no-host-metadata-property","severity":2,"message":"Use @HostBinding or @HostListener rather than the `host` metadata property (https://angular.io/styleguide#style-06-03)","line":53,"column":3,"nodeType":"Property","messageId":"noHostMetadataProperty","endLine":53,"endColumn":99}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/routing/wp-list-view/wp-primerized-list-view.component.ts#L85 <prefer-const>(https://eslint.org/docs/latest/rules/prefer-const)
'link' is never reassigned. Use 'const' instead.
Raw output
{"ruleId":"prefer-const","severity":2,"message":"'link' is never reassigned. Use 'const' instead.","line":85,"column":9,"nodeType":"Identifier","messageId":"useConst","endLine":85,"endColumn":13,"fix":{"range":[4004,4122],"text":"const link = this.pathHelper.workPackagePrimerDetailsPath(this.currentProject.identifier, workPackageId, tabIdentifier);"}}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/routing/wp-list-view/wp-primerized-list-view.component.ts#L125-L127 <padded-blocks>(https://eslint.org/docs/latest/rules/padded-blocks)
Block must not be padded by blank lines.
Raw output
{"ruleId":"padded-blocks","severity":2,"message":"Block must not be padded by blank lines.","line":125,"column":4,"nodeType":"ClassBody","messageId":"neverPadBlock","endLine":127,"endColumn":1,"fix":{"range":[5449,5451],"text":"\n"}}
github-actions / eslint
[eslint] frontend/src/app/features/work-packages/services/work-package-authorization.service.ts#L85 <@typescript-eslint/no-unsafe-argument>(https://typescript-eslint.io/rules/no-unsafe-argument)
Unsafe argument of type `any` assigned to a parameter of type `string | null`.
Raw output
{"ruleId":"@typescript-eslint/no-unsafe-argument","severity":2,"message":"Unsafe argument of type `any` assigned to a parameter of type `string | null`.","line":85,"column":48,"nodeType":"MemberExpression","messageId":"unsafeArgument","endLine":85,"endColumn":71}
github-actions / eslint
[eslint] frontend/src/app/shared/components/op-context-menu/handlers/op-types-context-menu.directive.ts#L120 <@typescript-eslint/no-floating-promises>(https://typescript-eslint.io/rules/no-floating-promises)
Promises must be awaited, end with a call to .catch, end with a call to .then with a rejection handler or be explicitly marked as ignored with the `void` operator.
Raw output
{"ruleId":"@typescript-eslint/no-floating-promises","severity":2,"message":"Promises must be awaited, end with a call to .catch, end with a call to .then with a rejection handler or be explicitly marked as ignored with the `void` operator.","line":120,"column":11,"nodeType":"ExpressionStatement","messageId":"floatingVoid","endLine":120,"endColumn":61,"suggestions":[{"messageId":"floatingFixVoid","fix":{"range":[4363,4363],"text":"void "},"desc":"Add void operator to ignore."}]}