-
Notifications
You must be signed in to change notification settings - Fork 517
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
Added alg_version details to test output #2080
Added alg_version details to test output #2080
Conversation
Signed-off-by: Pablo Gutiérrez <[email protected]>
Hi Pablo! The |
Signed-off-by: Pablo Gutiérrez <[email protected]>
Looks good to me! If you convert the PR to "Ready for review" I will give it an approving review. |
Restarted failing Travis CI to try to remove failing CI marker.... If green, proceed to merge. |
Merged. Thanks for the contribution @pablogf-uma ! |
This PR aims to fix #1786 by including the implementation path in the test_kem.c file for Kyber implementations. Before moving forward with other algorithms, I would like to confirm if this approach is correct and if the referenced implementation path aligns with the information requested in the issue.