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.

Sign up for GitHub

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

Jump to bottom

Update wc_KyberKey API to wc_MlKem #8469

Closed

devin-ai-integration wants to merge 1 commit into master from devin/1739917420-rename-kyberkey-to-mlkem

devin-ai-integration bot commented Feb 18, 2025

Update wc_KyberKey API to wc_MlKem

This PR renames the wc_KyberKey API to wc_MlKem for consistency with ML-KEM naming. Changes include:

Renamed all wc_KyberKey functions to wc_MlKem
Updated all function calls in tests and benchmarks
No behavioral changes, only renames
Maintained existing code style and documentation

Testing:

Changes are mechanical renames only
No local testing performed as this will be verified by CI
All function signatures and behavior remain identical

Link to Devin run: https://app.devin.ai/sessions/0912667958b540e8bc0e8a4b52070f42
Requested by: [email protected]


          Update wc_KyberKey API to wc_MlKem

9079a6d

Co-Authored-By: [email protected] <[email protected]>

Author

devin-ai-integration bot commented Feb 18, 2025

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

Disable automatic comment and CI monitoring

wolfSSL-Bot commented Feb 18, 2025

Can one of the admins verify this patch?

Contributor

SparkiDev commented Feb 18, 2025

Wrong API prefix

SparkiDev closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet