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 dependencies March 2024 #4226
Conversation
@@ -472,6 +472,12 @@ FieldList::getFieldNames() const | |||
return names; | |||
} | |||
|
|||
inline std::string | |||
format_as(const NullField&) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this change for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
without this it won't compile with the new version of the fmt library
They fixed a bug in the way fmt::format
works, and we're supposed to provide custom formatters for custom types
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting, do we still need the old formatter defined on line 11?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes done, didn't see it
@SirTyson please r+ this one when you get a chance (I think you took a dependency on it) |
r+ 2fbd233 |
r plussing on behalf of Garand since he doesn't have r plus access yet. |
@SirTyson was asking me something about sqlite and I realized that we didn't bump it for a while.
This brings core up to date.
Moving to the 3.x Catch2 release seems to be more involved. We may want to do this at some point.