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.

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

Remove GenericObject shim #2177

Merged
merged 1 commit into from Sep 29, 2022
Merged

Conversation

mrginglymus
Copy link
Contributor

This was declared in the local stubs file but not shipped, meaning that consumers with typescript libchecking enabled would have to also add the GenericObject definition to their local stubs file.

This commit inlines the definition where necessary, or replaces it with object where appropriate.

Fixes #2171

  • Code is linted
  • Tested
  • All new/updated code is covered with tests

This was declared in the local stubs file but not shipped, meaning that
consumers with typescript libchecking enabled would have to also add the
GenericObject definition to their local stubs file.

This commit inlines the definition where necessary, or replaces it with
object where appropriate.
@mrginglymus mrginglymus marked this pull request as ready for review September 29, 2022 08:23
@mrginglymus mrginglymus requested a review from a team as a code owner September 29, 2022 08:23
@mrginglymus mrginglymus requested review from AlexVarchuk and removed request for a team September 29, 2022 08:23
@AlexVarchuk
Copy link
Collaborator

AlexVarchuk commented Sep 29, 2022

Hi @mrginglymus, we very much appreciate your contribution. I'll retest that, and we include it in our next release.

@AlexVarchuk AlexVarchuk merged commit 127ef26 into Redocly:main Sep 29, 2022
@saskia-w
Copy link

Hi @AlexVarchuk : Is it known if this fix wil be released and when? Thanks in advanced

@mrginglymus mrginglymus deleted the generic-object branch February 13, 2023 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TS compilation error: Cannot find name 'GenericObject'
3 participants