Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Split queryF from query #694

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

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

Conversation

sigilion
Copy link

@sigilion sigilion commented Aug 6, 2020

In sigilion/purescript-run-halogen I find it useful to have a version of query that is not lifted into free, so that I can put the resulting HalogenF under VariantF in Run. I would prefer to have this split in the halogen source, rather than maintain my own version of query.

@thomashoneyman
Copy link
Member

I’m not necessarily against splitting out queryF (and the other constructors), but I also don’t think it’s a problem to maintain this separately downstream — I think it’s pretty rare to do that and this code doesn’t really change. I might like to wait for @garyb to have a chance to weigh in.

As a side note, I’m curious — what are you using run-Halogen for? Looks interesting!

@sigilion
Copy link
Author

run-halogen was split out of a side project I wrote to learn purescript. I don't think I can open source it; it was implementing the character generation process of the old pencil and paper RPG Traveler, and I think the process is copyrighted. The basic idea was to write the "business logic" using a bunch of domain specific algebraic effects (roll a die, choose from these options, enter a string etc.) plus State. I wrote a CLI version where these interactions are built around readline and the default state implementation, and a halogen version where the interactions are each delegated to a specific widget component and the state is the halogen state, so the page renders the state changes.

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.

2 participants