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

Partial revert of #1289 #1306

Merged
merged 1 commit into from
Aug 20, 2016
Merged

Conversation

adelbertc
Copy link
Contributor

All APIs that are present in 0.6.x continue to use Xor. New additions, notably FlatMapRec stuff, use Either.

All APIs that are present in 0.6.x continue to use `Xor`. New additions, notably FlatMapRec stuff, use `Either`.
@@ -251,7 +251,7 @@ import simulacrum.typeclass
/**
* Behaves like sequence_, but uses [[Unapply]] to find the
* [[Applicative]] instance for `G` - used when `G` is a
* type constructor with two or more parameters such as [[scala.util.Either]]
* type constructor with two or more parameters such as `scala.util.Either`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

For some reason validate was giving me "Could not find any members to link for "scala.util.Either" so I went for the easy way out.

Copy link
Contributor

Choose a reason for hiding this comment

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

I totally endorse this. I may-or-may-not have made a snarky tweet about Scaladoc today related to this.

@johnynek
Copy link
Contributor

👍
I did a grep of Either on this branch. It seems correct to me. All the remaining Either in core appear to be expects (tailRecM, EitherT, either syntax).

Thanks @adelbertc !

@codecov-io
Copy link

Current coverage is 90.60% (diff: 96.96%)

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

@@             master      #1306   diff @@
==========================================
  Files           235        235          
  Lines          3607       3608     +1   
  Methods        3549       3548     -1   
  Messages          0          0          
  Branches         54         56     +2   
==========================================
+ Hits           3268       3269     +1   
  Misses          339        339          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 6e12494...1867956

@non
Copy link
Contributor

non commented Aug 20, 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.

5 participants