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

Switch to visible type applications #56

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

Conversation

sigma-andex
Copy link
Contributor

Switch to visible type applications to make it easier to work with Variants

@jmatsushita
Copy link

jmatsushita commented Apr 12, 2024

Hey thanks, I was looking for this. However, since this approach breaks backward compatibility, how about instead introducing inj' prj' on' variants () that take VTAs?

If purescript-variant was ok with making a breaking release, then here's two libraries that would need to be fixed upstream (that I can identify in my codebase) are:

  • purescript-codec-argonaut that would also require a breaking change to allow variantCase to take a VTA as well.

EDIT: turns out it didn't garyb/purescript-codec-argonaut@master...jmatsushita:purescript-codec-argonaut:master

  • purescript-simple-json would also require a couple of patches here and here.

EDIT here's a patch for this too justinwoo/purescript-simple-json@master...jmatsushita:purescript-simple-json:master

@i-am-the-slime
Copy link

i-am-the-slime commented Aug 6, 2024

Not my library, but I hate function names with 's on them. For my own projects I have a VTA version of inj called as which I find much more beginner-friendly: as @"xyzException" 123
Or 123 # as @"xyzException".

@srghma
Copy link

srghma commented Oct 9, 2024

or inj prj on - new type applications
but injProxy prjProxy onProxy - old proxies way

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.

4 participants