-
Notifications
You must be signed in to change notification settings - Fork 96
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
Fix various MacOS build warnings #476
Conversation
I am getting the following error running the formatter manually. @ashman-p ➜ /workspaces/oqs-provider-fix-warnings (main) $ |
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.
LGTM, except for the upgraded minimum cmake version.
What version of |
This may pertain to #473 (also a |
Why did this get closed? The code looked like a valid fix and with the formatting update this should have passed CI, no? |
I was having problems getting coding formatting tests to pass on the branch. In my attempts to address that I sync'ed with your latest your changes in 'main'. That made things worst for the branch so, i decided to start over which closed the PR. Please stay tuned for a new PR. |
Address a few warning see on MacOS M1.
oqsprovider-MacOS-warnings.txt
Fixes #467.