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

Enforce define on Param paths #2

Open
chimon2000 opened this issue Aug 23, 2023 · 1 comment
Open

Enforce define on Param paths #2

chimon2000 opened this issue Aug 23, 2023 · 1 comment

Comments

@chimon2000
Copy link

One of the things that we have run into when using this library is that developers will occasionally forget to call define on paths that have a param in them. Is there a way to make this logic a bit safer?

It seems like it should be possible if path is removed from the Param class and define returns a Path object.

@mrgnhnt96
Copy link
Owner

Yeah, I've run into the same thing. There is an exception being thrown when path is called, which helps at runtime! But I agree, it'd be nice to have something alert you that you forgot about define.

I don't have time to work on this immediately. Though, I might be able to get to it within the coming weeks! If you have any suggestions or would like to fork the repo and create a PR, I'll welcome it!

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

No branches or pull requests

2 participants