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

feat(ci): test Kyber extraction and proofs #519

Merged
merged 2 commits into from
Feb 20, 2024
Merged

Conversation

W95Psp
Copy link
Collaborator

@W95Psp W95Psp commented Feb 20, 2024

This commit adds to CI the extraction to F* of Kyber and TC.
This mostly is a copy-paste from https://github.com/cryspen/libcrux/blob/main/.github/workflows/hax.yml.

Note this job is a bit long, thus it is ran only on when merging in the merge queue.

Status: this is working, but currently Kyber TC is broken, so we need cryspen/libcrux#202 to be merged first

@W95Psp W95Psp force-pushed the gha-add-kyber-workflow branch 2 times, most recently from bd26dac to 1c70079 Compare February 20, 2024 10:59
@W95Psp W95Psp marked this pull request as ready for review February 20, 2024 11:36
Copy link
Member

@franziskuskiefer franziskuskiefer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like some debug leftovers are still in here, otherwise this looks good to me.

.github/workflows/kyber.yml Outdated Show resolved Hide resolved
.github/workflows/kyber.yml Outdated Show resolved Hide resolved
.github/workflows/kyber.yml Show resolved Hide resolved
W95Psp and others added 2 commits February 20, 2024 14:05
This commit adds to CI the extraction to F* of Kyber and TC.

Note this job is a bit long, thus it is ran only on when merging in the merge queue.
Co-authored-by: Franziskus Kiefer <[email protected]>
@W95Psp W95Psp added this pull request to the merge queue Feb 20, 2024
Merged via the queue into main with commit 6473c1e Feb 20, 2024
11 checks passed
@W95Psp W95Psp deleted the gha-add-kyber-workflow branch February 20, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants