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

Clarify some details about the life and death of Xor #1428

Merged
merged 1 commit into from Oct 25, 2016

Conversation

@travisbrown
Copy link
Collaborator

@travisbrown travisbrown commented Oct 25, 2016

These changes are minor, but to my ear "up to Cats 0.6.x" sounds like it excludes 0.6.x, and in any case Xor wasn't actually gone in 0.7. Since we're pointing to this FAQ entry in the 0.8.0 release / upgrade notes, I thought we should make the timing a little clearer. /cc @adelbertc

@travisbrown travisbrown mentioned this pull request Oct 25, 2016
13 of 13 tasks complete
@kailuowang
Copy link
Member

@kailuowang kailuowang commented Oct 25, 2016

👍

@adelbertc
Copy link
Member

@adelbertc adelbertc commented Oct 25, 2016

👍 on Travis green

@codecov-io
Copy link

@codecov-io codecov-io commented Oct 25, 2016

Current coverage is 92.20% (diff: 100%)

Merging #1428 into master will not change coverage

@@             master      #1428   diff @@
==========================================
  Files           242        242          
  Lines          3606       3606          
  Methods        3537       3537          
  Messages          0          0          
  Branches         69         69          
==========================================
  Hits           3325       3325          
  Misses          281        281          
  Partials          0          0          

Powered by Codecov. Last update 93a4d6a...6b1b7fa

@peterneyens peterneyens merged commit 35c2bc0 into typelevel:master Oct 25, 2016
3 checks passed
3 checks passed
codecov/patch Coverage not affected when comparing 93a4d6a...6b1b7fa
Details
codecov/project 92.20% (+0.00%) compared to 93a4d6a
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@stew stew removed the in progress label Oct 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.