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

Add explicit Id instances for Kleisli #621

Merged
merged 2 commits into from
Nov 12, 2015

Conversation

ceedubs
Copy link
Contributor

@ceedubs ceedubs commented Nov 10, 2015

Fixes #580.

@codecov-io
Copy link

Current coverage is 76.22%

Merging #621 into master will not affect coverage as of 57ab26c

@@            master    #621   diff @@
======================================
  Files          159     159       
  Stmts         2074    2078     +4
  Branches        69      69       
  Methods          0       0       
======================================
+ Hit           1581    1584     +3
  Partial          0       0       
- Missed         493     494     +1

Review entire Coverage Diff as of 57ab26c

Powered by Codecov. Updated on successful CI builds.

@adelbertc
Copy link
Contributor

👍

@non
Copy link
Contributor

non commented Nov 12, 2015

👍 but needs a re-merge

@ceedubs
Copy link
Contributor Author

ceedubs commented Nov 12, 2015

Thanks @non. Updated.

non added a commit that referenced this pull request Nov 12, 2015
Add explicit Id instances for Kleisli
@non non merged commit c4626ee into typelevel:master Nov 12, 2015
@non non removed the in progress label Nov 12, 2015
@ceedubs ceedubs deleted the kleisli-id-instances branch November 15, 2015 23:38
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.

4 participants