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

supplement missing types of tools #1056

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

Conversation

sunshinego12138
Copy link

I found that I cannot get the correct type prompt for the third parameter of the request method, perhaps because the type tool is missing #995

@SaltyAom
Copy link
Member

Hi, thanks for submitting the pull request.
I assume that you're trying to add NoInfer utility type to Elysia?
If this was correct, TypeScript has added NoInfer type natively since version 5.4 which has much more performance overhead than the user-defined one.
So we're relying on TypeScript for that feature.

I suggest is it possible if you could try to update TypeScript to at least 5.4 and check if the code work?

It would be great to stick to the native TypeScript functionality, thanks.

@SaltyAom SaltyAom self-assigned this Feb 16, 2025
@SaltyAom SaltyAom added the question Further information is requested label Feb 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants