Skip to content

Conversation

@damncabbage
Copy link
Contributor

@damncabbage damncabbage commented Dec 29, 2016

This adds /\ as a type operator alias, and corrects (I think...?) a comment on the value-level operator alias while I was at it.

With no tests in the repo, I've tested by swapping out some of the T2, T3 type aliases with /\s, and it worked out. I've left those uncommitted, but I'm happy to add them in or make other changes if you think they'd be useful.

(This should close #24.)

@paf31
Copy link
Contributor

paf31 commented Dec 29, 2016

Seems reasonable to me. @garyb any thoughts?

@garyb
Copy link
Member

garyb commented Dec 29, 2016

Looks good to me!

@garyb garyb merged commit 55cbdef into purescript:master Dec 29, 2016
@damncabbage damncabbage deleted the topic/type-level-operator branch December 30, 2016 06:03
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.

Add type operator for /\

3 participants