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

update the logging submodule to remove warnings #131

Merged
merged 8 commits into from
Nov 15, 2023
Merged

Conversation

kevinkreiser
Copy link
Owner

as was done in valhalla

@kevinkreiser
Copy link
Owner Author

the build system for mac changed. i updated the project ot make use of it, got it to the point of passing the build stage but a few things are wrong.

  • the newer compiler is hyper pedantic
  • 3 of the tests fail

we will fix them in another pr

@kevinkreiser kevinkreiser merged commit b7ca819 into master Nov 15, 2023
1 of 2 checks passed
@kevinkreiser kevinkreiser deleted the kk_logging branch November 15, 2023 00:00
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.

1 participant