Skip to content

Compilation flag with zcash's implementation #254

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

Merged
merged 7 commits into from
Jun 8, 2022
Merged

Conversation

iquerejeta
Copy link
Contributor

Using zcash's backend seems to resolve #207 . We change mithril-core so that it can be compiled with a flag choosing between one implementation or the other. Preferably, we want to use blst in production, but only if this error is resolved.

@github-actions
Copy link

github-actions bot commented Jun 8, 2022

Unit Test Results

186 tests  +1   186 ✔️ +1   8m 34s ⏱️ + 6m 24s
  14 suites ±0       0 💤 ±0 
    5 files   ±0       0 ±0 

Results for commit cf5d3ff. ± Comparison against base commit 23abfd2.

♻️ This comment has been updated with latest results.

@iquerejeta iquerejeta mentioned this pull request Jun 8, 2022
@iquerejeta iquerejeta marked this pull request as ready for review June 8, 2022 11:14
@iquerejeta iquerejeta requested review from jpraynaud and Alenar June 8, 2022 11:14
Copy link
Member

@jpraynaud jpraynaud left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Such good news if it fixes the flaky tests 🥳

@iquerejeta iquerejeta merged commit c9213ca into main Jun 8, 2022
@iquerejeta iquerejeta deleted the backend_flag branch June 8, 2022 12:38
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.

Flaky Tests CI
2 participants