-
Notifications
You must be signed in to change notification settings - Fork 29
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
Consider giving the Y-combinator, fix
, two type parameters.
#29
Comments
Hm, it's an interesting proposal. What use cases have you run into where that version is more useful? N.B. I kept |
I'm in favor of this proposal. It's strictly more general without changing any of the existing uses. |
Hi CodaFi, thanks for the quick response. I'm trying out parser combinators, and loving it. I was using robrix's Bit of a newbie, but you can see an example of my use case here I'd be happy to create a PR if desired. Will a |
@DanielAsher Would you be up for submitting a pull request, or shall I do it? |
@CodaFi I be rather excited to submit a PR - it would be my first! I'd like to check out |
@CodaFi @pthariensflame thanks so much for quick and insightful responses 👍 |
Currently
fix
is defined so:I have found the following
fix
more useful:Could this act as a replacement, or addition, to the current
fix
?cheers,
Daniel
The text was updated successfully, but these errors were encountered: