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

Back to either #1310

Merged
merged 2 commits into from
Aug 22, 2016
Merged

Back to either #1310

merged 2 commits into from
Aug 22, 2016

Conversation

adelbertc
Copy link
Contributor

@adelbertc adelbertc commented Aug 21, 2016

  • Remove all traces of Xor and XorT
  • Add a section to the FAQ talking about Either syntax
  • Move Xor tut to Either and update tuts to reflect the change

@adelbertc
Copy link
Contributor Author

/cc @non @johnynek @travisbrown

@codecov-io
Copy link

Current coverage is 90.61% (diff: 96.77%)

Merging #1310 into master will decrease coverage by 0.01%

@@             master      #1310   diff @@
==========================================
  Files           237        233     -4   
  Lines          3725       3496   -229   
  Methods        3663       3442   -221   
  Messages          0          0          
  Branches         58         54     -4   
==========================================
- Hits           3376       3168   -208   
+ Misses          349        328    -21   
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 21fb277...2afeed8

@non
Copy link
Contributor

non commented Aug 21, 2016

LGTM 👍

@johnynek
Copy link
Contributor

👍

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.

6 participants