Skip to content
This repository was archived by the owner on Oct 4, 2020. It is now read-only.

Add semigroup and monoid instances #25

Merged
merged 2 commits into from
Dec 8, 2014
Merged

Add semigroup and monoid instances #25

merged 2 commits into from
Dec 8, 2014

Conversation

hdgarrood
Copy link
Contributor

As mentioned in #23

@paf31
Copy link
Contributor

paf31 commented Dec 8, 2014

Looks good to me. @garyb @joneshf Any comments?

@hdgarrood
Copy link
Contributor Author

Oh - might we also want a Monoid instance where mempty = empty?

@paf31
Copy link
Contributor

paf31 commented Dec 8, 2014

I think so.

@garyb
Copy link
Member

garyb commented Dec 8, 2014

👍 for both.

@hdgarrood hdgarrood changed the title Add semigroup instance Add semigroup and monoid instances Dec 8, 2014
@hdgarrood
Copy link
Contributor Author

done :)

paf31 added a commit that referenced this pull request Dec 8, 2014
Add semigroup and monoid instances
@paf31 paf31 merged commit ebc86a6 into purescript-deprecated:master Dec 8, 2014
@paf31
Copy link
Contributor

paf31 commented Dec 8, 2014

👍 Thanks!

@hdgarrood hdgarrood deleted the semigroup-map branch December 8, 2014 21:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants