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

feat: infer type of the URLPattern context passed to the handler #12

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Danielduel
Copy link

@Danielduel Danielduel commented Mar 22, 2024

Hi, I think the change is very visible when you open /examples/typesafe_parameters.ts.

I have few things missing in the typesafety of the internals - like few nevers that shouldn't be nevers and cleaning up few snippets that I was using to beat the typesystem to quickly check if everything is bueno.

I open this to get initial feedback if it is welcome or I should go and create something like safe fork packaged under different name (which is less preferred option for me).

I might add this to URLPattern itself in future, I will migrate rutt if my changes will get there, I felt like changing rutt will be far easier because the narrow scope of the lib.

Thanks!

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.

1 participant