issue#8 added Day Type from time library#21
Conversation
|
Thank you! Looks good. As you might have noticed there are two overlapping pieces of API: the 'getX' functions and the Traversal s. I like traversals better as they are more flexible but this is a longer-term design question, to be explored with concrete examples. |
|
I've added some additional types now. |
|
Sorry, I've noticed that the checks in the CI failed, but I don't know why that is. It builds and tests pass locally on my machine. |
|
@AR2202 the CI error is due to the version bound on the Error: While constructing the build plan, the following exceptions were encountered: In the dependencies for heidi-0.3.1: needed since heidi is a build target. Some different approaches to resolving this:
|
|
Thanks @ocramz for your explanation. |
I added the 'Day' type from the time library, which represents a Date. This is a first attempt and if this is OK, I can add more types from the time library.