-
Notifications
You must be signed in to change notification settings - Fork 79
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
Automatic conversion to scala.concurrent.duration #54
Comments
Oh I was terribly wrong! val x = 3 seconds // type inference says it's a com.github.nscala_time.time.DurationBuilder So I need an implicit conversion from DurationBuilder, but it's a private class! So the following just won't compile package implicit_vs_private
import scala.concurrent.duration.{Duration => ScalaDuration}
import com.github.nscala_time.time.Imports._
import com.github.nscala_time.time.DurationBuilder
object Test {
implicit def NscalaDurationBuilder2Duration(x: DurationBuilder) : ScalaDuration = ScalaDuration.fromNanos(x.standardDuration.getMillis * 1000000)
} |
Hi. There are another request to make What do you think? /cc: @gakuzzzz @xuwei-k @tototoshi |
I'm your side, @kmizu. I's not so good to make it public |
Today, I decided to make |
Fixed via #64 |
What you think about to include @kepkin conversion's into nscala-time? I think it would be very useful. |
@yokomizor OK. I think it's a good idea to add @kepkin conversion's into nscala-time. Since I'm a little busy, it helps us if you create the PR to add @kepkin conversion's into nscala-time. Or please wait a while for creating the PR. |
@yokomizor Note that Since nscala-time is simple sbt project, I think that you can edit nscala-time code easily via text editors or IDEs (Especially, IntelliJ IDEA + Scala Plugin). |
@yokomizor I'm sorry to late response. After I create the PR #69 and the PR's badness is point outed, I found it bad idea that @kepkin conversions are included into nscala-time. Then, I close this issue. |
Hi!
I'm using nscala-time with RxScala, so I had to write a simple implicit conversion:
I think it's possible to include this conversion into nscala-time
The text was updated successfully, but these errors were encountered: