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.

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

Implemented the stream operator<< for type datetime #1314

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PBRCW
Copy link

@PBRCW PBRCW commented Jan 20, 2020

This change enables the usage of type datetime as a key e. g. in an OData RESTful API implementation (Like the OData C++ Client Library that is build on top of Casablanca). Without this operator overload the attempt to do so would fail.

…ng datetime as a key e. g. in an OData RESTful API implementation.
@msftclas
Copy link

msftclas commented Jan 20, 2020

CLA assistant check
All CLA requirements met.

@BillyONeal
Copy link
Member

I think this is OK but you need to add a test before we would want to merge this.

@sebkraemer
Copy link

sebkraemer commented May 12, 2021

I think this is OK but you need to add a test before we would want to merge this.

Ping. Could this PR (and the mentioned follow-up PR with tests) be merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants