Skip to content

Conversation

@Thimoteus
Copy link
Contributor

No description provided.

@paf31
Copy link
Contributor

paf31 commented Oct 2, 2017

👍 Makes sense to me

@garyb
Copy link
Member

garyb commented Oct 2, 2017

I know it's exactly the same thing, but wouldn't people usually say apply = ap (since you define apply for the instance but ap is derived)? Either way 💯

@Thimoteus
Copy link
Contributor Author

Ah. I was thinking putting ap on the LHS since it's defined in the same module. Something about reading left-to-right, seeing ap, asking "wait, what's that? oh, it's a few lines below," then reading = apply.

@garyb garyb merged commit da4e752 into purescript:master Jan 31, 2018
@garyb
Copy link
Member

garyb commented Jan 31, 2018

Thanks! Forgot about this one until a relevant discussion popped up today 😉

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.

3 participants