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.

Sign up for GitHub

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

Jump to bottom

Fixes type params for OptionT.getOrElse(F) #2119

Merged
merged 3 commits into from
Jan 22, 2018
Merged

Fixes type params for OptionT.getOrElse(F) #2119

merged 3 commits into from
Jan 22, 2018

Commits on Dec 18, 2017

  1. Fixes type params for OptionT.getOrElse(F) 

    Adds a type parameter to getOrElse and getOrElseF to conform to
Option.getOrElse, which allows it to be called without needing a type
hint:

OptionT(Future {
  Option(Right(1))
}).getOrElse(Left("error"))

Instead of:

OptionT(Future {
  Option(Right(1): Either[String, Int])
}).getOrElse(Left("error"))
    @fcanedo
    fcanedo committed Dec 18, 2017
    Configuration menu
    Copy the full SHA
    7ccaa4f View commit details
    Browse the repository at this point in the history

Commits on Dec 21, 2017

  1. Adds some tests

    @fcanedo
    fcanedo committed Dec 21, 2017
    Configuration menu
    Copy the full SHA
    a9c118e View commit details
    Browse the repository at this point in the history

Commits on Jan 13, 2018

  1. trying to fix build for 2.11-

    @kailuowang
    kailuowang committed Jan 13, 2018
    Configuration menu
    Copy the full SHA
    cb2aba4 View commit details
    Browse the repository at this point in the history