Skip to content

add ap = apply law #141

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jan 31, 2018
Merged

add ap = apply law #141

merged 1 commit into from
Jan 31, 2018

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 😉

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants