Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#59037] Primerize "Log time" dialog #17268

Draft
wants to merge 58 commits into
base: dev
Choose a base branch
from

fix 2 issues in the dialog

79a82db
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Draft

[#59037] Primerize "Log time" dialog #17268

fix 2 issues in the dialog
79a82db
Select commit
Loading
Failed to load commit list.
GitHub Actions / eslint failed Dec 23, 2024 in 0s

reviewdog [eslint] report

reported by reviewdog 🐶

Findings (2)

frontend/src/app/features/calendar/te-calendar/te-calendar.component.ts|597 col 9| Unsafe argument of type any assigned to a parameter of type TimeEntryResource.
frontend/src/app/shared/components/op-context-menu/wp-context-menu/wp-single-context-menu.ts|28 col 14| 'HookService' is already declared in the upper scope on line 9 column 10.

Filtered Findings (0)

Annotations

Check failure on line 597 in frontend/src/app/features/calendar/te-calendar/te-calendar.component.ts

See this annotation in the file changed.

@github-actions github-actions / eslint

[eslint] frontend/src/app/features/calendar/te-calendar/te-calendar.component.ts#L597 <@typescript-eslint/no-unsafe-argument>(https://typescript-eslint.io/rules/no-unsafe-argument) 

Unsafe argument of type `any` assigned to a parameter of type `TimeEntryResource`.
Raw output 
{"ruleId":"@typescript-eslint/no-unsafe-argument","severity":2,"message":"Unsafe argument of type `any` assigned to a parameter of type `TimeEntryResource`.","line":597,"column":9,"nodeType":"MemberExpression","messageId":"unsafeArgument","endLine":597,"endColumn":40}

Check failure on line 28 in frontend/src/app/shared/components/op-context-menu/wp-context-menu/wp-single-context-menu.ts

See this annotation in the file changed.

@github-actions github-actions / eslint

[eslint] frontend/src/app/shared/components/op-context-menu/wp-context-menu/wp-single-context-menu.ts#L28 <@typescript-eslint/no-shadow>(https://typescript-eslint.io/rules/no-shadow) 

'HookService' is already declared in the upper scope on line 9 column 10.
Raw output 
{"ruleId":"@typescript-eslint/no-shadow","severity":2,"message":"'HookService' is already declared in the upper scope on line 9 column 10.","line":28,"column":14,"nodeType":"Identifier","messageId":"noShadow","endLine":28,"endColumn":37}
View more details on GitHub Actions