KEMBAR78
implement year_month_day by zivshek · Pull Request #3913 · fmtlib/fmt · GitHub
Skip to content

Conversation

@zivshek
Copy link
Contributor

@zivshek zivshek commented Mar 30, 2024

also changed weekday, day, month, year's formatter to use formatter<std::tm, Char> so they all support the format strings

fixes #3772

Copy link
Contributor

@vitaut vitaut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

@zivshek zivshek requested a review from vitaut March 31, 2024 02:48
@zivshek zivshek requested a review from vitaut April 1, 2024 13:34
@zivshek
Copy link
Contributor Author

zivshek commented Apr 3, 2024

@vitaut in case you missed the re-review request

@vitaut vitaut merged commit 71144ee into fmtlib:master Apr 3, 2024
@vitaut
Copy link
Contributor

vitaut commented Apr 3, 2024

Merged, thanks!

happymonkey1 pushed a commit to happymonkey1/fmt that referenced this pull request Apr 7, 2024
* implement year_month_day, also changed weekday, day, month, year's formatter to use formatter<std::tm, Char> so they all support the format strings
* support ":L" for month and weekday
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for std::chrono::year_month_day

3 participants