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

Tests: also include the crypto_kem directory #12

Merged
merged 1 commit into from
Oct 4, 2022

Conversation

vbgl
Copy link
Collaborator

@vbgl vbgl commented Sep 27, 2022

Do not merge before jasmin-lang/jasmin#250 is fixed and jasmin-lang/jasmin#249 is merged.

@vbgl
Copy link
Collaborator Author

vbgl commented Oct 3, 2022

When is the jasmin version updated in the CI machine? I believed that CI should be green by now.

@tfaoliveira
Copy link
Member

When is the jasmin version updated in the CI machine? I believed that CI should be green by now.

Hi @vbgl, the update should happen soon, I'm still catching up, but I think I should use https://github.com/jasmin-lang/jasmin/releases/tag/v2022.09.0

@vbgl
Copy link
Collaborator Author

vbgl commented Oct 4, 2022

Best if you can test against released versions (aka 2022.09.0) and development versions (aka main).

@tfaoliveira
Copy link
Member

Best if you can test against released versions (aka 2022.09.0) and development versions (aka main).

OK. Should I consider each commit in Jasmin main? Or once per day would be fine?

@tfaoliveira
Copy link
Member

I see that build-from-tarball is quite fast, so, for each commit then

@tfaoliveira tfaoliveira merged commit 04f58b3 into main Oct 4, 2022
@vbgl vbgl deleted the ec-check-crypto_kem branch October 4, 2022 11:17
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.

2 participants