Skip to content

Conversation

@d-winsor
Copy link
Contributor

No description provided.

@d-winsor d-winsor requested a review from a team as a code owner March 22, 2021 20:38
@StephanTLavavej StephanTLavavej added chrono C++20 chrono cxx20 C++20 feature labels Mar 22, 2021
@StephanTLavavej StephanTLavavej merged commit 74abbc2 into microsoft:feature/chrono Mar 23, 2021
@StephanTLavavej
Copy link
Member

As the tests passed, I went ahead and merged this - please let me know if I messed anything up during review (particularly using the previously-unused _Version). 😸

@d-winsor
Copy link
Contributor Author

As the tests passed, I went ahead and merged this - please let me know if I messed anything up during review (particularly using the previously-unused _Version). 😸

Looks good to me and that was absolutely a bug :). Just didn't catch it because I would have had to use test hooks to test that pass.

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

Labels

chrono C++20 chrono cxx20 C++20 feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants