Skip to content
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

Rename SemigroupK.combine to combineK #781

Merged
merged 1 commit into from
Jan 30, 2016
Merged

Conversation

ceedubs
Copy link
Contributor

@ceedubs ceedubs commented Jan 6, 2016

Fixes #763.

@codecov-io
Copy link

Current coverage is 88.47%

Merging #781 into master will not affect coverage as of 3e37007

@@            master    #781   diff @@
======================================
  Files          166     166       
  Stmts         2291    2291       
  Branches        75      75       
  Methods          0       0       
======================================
  Hit           2027    2027       
  Partial          0       0       
  Missed         264     264       

Review entire Coverage Diff as of 3e37007

Powered by Codecov. Updated on successful CI builds.

@ceedubs
Copy link
Contributor Author

ceedubs commented Jan 30, 2016

@travisbrown since you mentioned that you'd like to see this in the next Cats release, could you be so kind as to review it? :)

@mpilquist
Copy link
Member

👍

1 similar comment
@travisbrown
Copy link
Contributor

👍

ceedubs added a commit that referenced this pull request Jan 30, 2016
Rename SemigroupK.combine to combineK
@ceedubs ceedubs merged commit 33642a0 into typelevel:master Jan 30, 2016
@ceedubs ceedubs deleted the combineK branch January 30, 2016 15:44
# 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.

4 participants