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

remove Unapply fixes #1607 #1679

Merged
merged 2 commits into from
May 18, 2017
Merged

Conversation

kailuowang
Copy link
Contributor

It seems that everyone agrees that Unapply shouldn't exist in cats.core any more. I propose we remove it first and we can find a place for it later if we need to. Leaving in cats may cause some confusion. e.g. people may still use it to add xxxxU versions, etc.

@codecov-io
Copy link

Codecov Report

Merging #1679 into master will increase coverage by 0.41%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1679      +/-   ##
==========================================
+ Coverage   93.42%   93.83%   +0.41%     
==========================================
  Files         242      241       -1     
  Lines        4075     4055      -20     
  Branches      140      139       -1     
==========================================
- Hits         3807     3805       -2     
+ Misses        268      250      -18

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6ad7d8f...96c4c70. Read the comment docs.

@kailuowang kailuowang merged commit a5348cf into typelevel:master May 18, 2017
@kailuowang kailuowang deleted the remove-unapply branch May 18, 2017 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants