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

Jump to bottom

style(api-client): fix Unexpected any linter warnings in extendWith<T> #2675

Closed
petermetz opened this issue Sep 10, 2023 · 3 comments · Fixed by #2698
Closed

style(api-client): fix Unexpected any linter warnings in extendWith<T> #2675

petermetz opened this issue Sep 10, 2023 · 3 comments · Fixed by #2698
Assignees
@bmvd70
Labels
backlog Tasks that are needed but we cannot prioritize them at present. enhancement New feature or request good-first-issue Good for newcomers good-first-issue-100-introductory Hacktoberfest Hacktoberfest participants are welcome to take a stab at issues marked with this label. P5 Priority 5: Lowest
Milestone
v2.0.0

Comments

@petermetz
Copy link
Contributor

Description

image
@petermetz petermetz added enhancement New feature or request good-first-issue Good for newcomers Hacktoberfest Hacktoberfest participants are welcome to take a stab at issues marked with this label. good-first-issue-100-introductory P5 Priority 5: Lowest backlog Tasks that are needed but we cannot prioritize them at present. labels Sep 10, 2023
@petermetz petermetz added this to the vT.B.D milestone Sep 10, 2023
@petermetz
Copy link
Contributor Author

Pro tip: Use Record<string, unknown> as the type for these variables insted of any.

This was referenced Sep 10, 2023
Closed
Closed
Closed
Closed
Closed
Closed
Closed
@bmvd70
Copy link
Contributor

bmvd70 commented Sep 17, 2023

I'd like to fix this issue

bmvd70 added a commit to bmvd70/cacti that referenced this issue Sep 17, 2023
@bmvd70
style: fix unexpected any linter warnings in extendWithT
64a2836 
Used the suggester Record<string, unknown> to replace any

Closes: hyperledger-cacti#2675
Signed-off-by: bmvd70 <bastian.vidal@mni.thm.de>
@bmvd70 bmvd70 mentioned this issue Sep 17, 2023
Merged
@petermetz petermetz modified the milestones: vT.B.D, v2.0.0 Sep 18, 2023
@petermetz
Copy link
Contributor Author

@bmvd70 Nice, thank you! Assigned it just now.

sandeepnRES pushed a commit to sandeepnRES/cacti that referenced this issue Dec 21, 2023
@bmvd70 @sandeepnRES
style: fix unexpected any linter warnings in extendWithT
0107e98 
Used the suggester Record<string, unknown> to replace any

[skip ci]

Closes: hyperledger-cacti#2675

Signed-off-by: bmvd70 <bastian.vidal@mni.thm.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bmvd70 bmvd70

Labels
backlog Tasks that are needed but we cannot prioritize them at present. enhancement New feature or request good-first-issue Good for newcomers good-first-issue-100-introductory Hacktoberfest Hacktoberfest participants are welcome to take a stab at issues marked with this label. P5 Priority 5: Lowest
Projects
None yet
Milestone
v2.0.0
Development

Successfully merging a pull request may close this issue.

style: fix unexpected any linter warnings in extendWithT bmvd70/cacti
2 participants
@petermetz @bmvd70