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

Bump libfmt; enable Qt 6 build; add std::format support #29

Merged
merged 2 commits into from
Mar 25, 2024

Conversation

dangelog
Copy link
Contributor

@dangelog dangelog commented Mar 5, 2024

No description provided.

@dangelog dangelog changed the title Bump libfmt and enable Qt 6 build Bump libfmt; enable Qt 6 build; add std::format support Mar 5, 2024
@dangelog dangelog requested review from narnaud and mgiroday March 5, 2024 12:25
@narnaud
Copy link
Contributor

narnaud commented Mar 5, 2024

I don't know all the details of writing a std formatter, and didn't try the code yet, but I couldn't catch any issue.
The unit-tests seems all cases have been handled.

Copy link
Contributor

@mgiroday mgiroday left a comment

Choose a reason for hiding this comment

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

Tests works fine for me. Did not try the library real life yet.

@dangelog
Copy link
Contributor Author

dangelog commented Mar 6, 2024

@winterz is it OK to merge?

@winterz winterz merged commit 572b79d into master Mar 25, 2024
6 checks passed
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.

4 participants