Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upMove Time type alias(es) to Basics module #415
Conversation
jvoigtlaender
referenced this pull request
Sep 29, 2015
Closed
Time type alias defined in different places #412
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
jvoigtlaender
Oct 7, 2015
Contributor
Maybe do this now, so it doesn't need a jump for core from 3.0.0 to 4.0.0 too soon?
|
Maybe do this now, so it doesn't need a jump for core from 3.0.0 to 4.0.0 too soon? |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
jvoigtlaender
Feb 5, 2016
Contributor
@evancz: An additional concrete benefit of this change (apart from what we discussed in the other thread linked above) would be that Time in the type signature at http://package.elm-lang.org/packages/elm-lang/core/latest/Task#sleep would then properly link to a definition of Time (which currently it does not).
|
@evancz: An additional concrete benefit of this change (apart from what we discussed in the other thread linked above) would be that |
jvoigtlaender commentedSep 29, 2015
Follow-up to the discussion in https://github.com/elm-lang/core/issues/412.