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

fix: Generic static This. #2533

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

max-kahnt-keylight
Copy link
Contributor

Attempt to fix #2531.
Should probably be extended to be used in all static This inferences.

@1mike12
Copy link
Contributor

1mike12 commented Jul 22, 2024

i know this is a while ago, but can you give an example of what was broken before? Since this is not maintained anymore, I am ingesting all fixes into my fork. This seems similar to lots of other typescript fixes that were already proposed and merged into my fork. Woudl love to add your fix too

@max-kahnt-keylight
Copy link
Contributor Author

I am not sure what additional context to give apart from the issue #2531 that is already linked.
It seems like this fix is/was needed in order to have the changes from #2399 (which has been reverted).

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.

Type inferred is wrong when using model constructor
2 participants