boxdot / openapi-schema

Apache License 2.0
4 stars 1 forks source link

Add the support of chrono::DateTime and chrono::Date #2

Closed antoine-de closed 5 years ago

antoine-de commented 5 years ago

Since it brings another dependency, I've made it optional, I don't know what you think about this

should we enable it by default ?

boxdot commented 5 years ago

I think you approach with feature chrono is good. Since datetime is fundamental for REST apis, I think this should be enabled by default, like you did.

Thanks for contribution!