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

added a migration note to the release notes IRT tuple instances #1009

Merged
merged 4 commits into from
Apr 30, 2016

Conversation

kailuowang
Copy link
Contributor

No description provided.

@codecov-io
Copy link

codecov-io commented Apr 28, 2016

Current coverage is 83.92%

Merging #1009 into master will increase coverage by +<.01%

  1. File ...ats/std/anyval.scala (not in diff) was modified. more
    • Misses -1
    • Partials 0
    • Hits +1
  2. File ...ats/std/option.scala (not in diff) was modified. more
    • Misses -1
    • Partials 0
    • Hits +1
@@             master      #1009   diff @@
==========================================
  Files           368        368          
  Lines          2376       2376          
  Methods        2169       2169          
  Messages          0          0          
  Branches         19         19          
==========================================
+ Hits           1992       1994     +2   
+ Misses          384        382     -2   
  Partials          0          0          

Powered by Codecov. Last updated by 9f76be1

@@ -6,10 +6,12 @@ Version 0.5.0 is the fifth release.

This release includes some API changes:

`cats.laws.discipline.eq` no longer provides `Eq` instances for `tuple2` and `tuple3`, these instances and together with some other new instances for `tuple`s are not provided by `cats.std.tuple` (through inheriting the instance trait defined in algebra 0.4.2).
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a typo right? Should be "are now provided by" right?

@@ -6,10 +6,12 @@ Version 0.5.0 is the fifth release.

This release includes some API changes:

`cats.laws.discipline.eq` no longer provides `Eq` instances for `tuple2` and `tuple3`, these instances and together with some other new instances for `tuple`s are now provided by `cats.std.tuple` (through inheriting the instance trait defined in algebra 0.4.2).
Copy link
Contributor

Choose a reason for hiding this comment

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

Nitpicky but could you capitalize Tuple2, etc.? The formatting and context makes it look like you're referring to real types.

@travisbrown
Copy link
Contributor

👍, thanks @kailuowang!

@non
Copy link
Contributor

non commented Apr 30, 2016

👍 Thanks!

@non non merged commit dfc0d6f into typelevel:master Apr 30, 2016
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