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

use simulacrum everywhere possible #294

Merged
merged 4 commits into from May 2, 2015
Merged

use simulacrum everywhere possible #294

merged 4 commits into from May 2, 2015

Conversation

stew
Copy link
Contributor

@stew stew commented Apr 25, 2015

No description provided.

@@ -11,7 +11,7 @@ import simulacrum._
* `foldLeft` methods, which will usually be used by the associationed
* `Fold[_]` instance.
*
* Foldable[F] is implemented in terms of two basic methods:
b * Foldable[F] is implemented in terms of two basic methods:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like a typo

@non
Copy link
Contributor

non commented Apr 26, 2015

Once you fix that typo, 👍 from me.

@stew
Copy link
Contributor Author

stew commented Apr 26, 2015

but wait, there's more :)

@stew
Copy link
Contributor Author

stew commented Apr 26, 2015

oops, I have tests fix

@japgolly
Copy link

Very cool.

@ceedubs
Copy link
Contributor

ceedubs commented Apr 26, 2015

Looking good so far! The build is showing quite a few compile errors in the laws, though.

@dwijnand
Copy link
Contributor

Nice work.

some kind of macro expansion problem
@ceedubs
Copy link
Contributor

ceedubs commented May 2, 2015

👍

@non
Copy link
Contributor

non commented May 2, 2015

great 👍

non added a commit that referenced this pull request May 2, 2015
use simulacrum everywhere possible
@non non merged commit ecf0a03 into typelevel:master May 2, 2015
@non non removed the in progress label May 2, 2015
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.

None yet

5 participants