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

Use Nix for building and testing #92

Merged
merged 1 commit into from
Jul 10, 2024
Merged

Use Nix for building and testing #92

merged 1 commit into from
Jul 10, 2024

Conversation

x-mass
Copy link
Member

@x-mass x-mass commented Jun 18, 2024

No description provided.

@x-mass x-mass force-pushed the nix branch 15 times, most recently from 5562acc to 79a7f16 Compare June 24, 2024 20:38
@x-mass x-mass force-pushed the nix branch 5 times, most recently from 415d53a to 0c0031f Compare June 27, 2024 19:52
@x-mass x-mass marked this pull request as ready for review July 8, 2024 21:12
@x-mass x-mass requested a review from martun July 8, 2024 21:12
@x-mass x-mass force-pushed the nix branch 5 times, most recently from ae22ea5 to ac8fe22 Compare July 10, 2024 10:21
@x-mass
Copy link
Member Author

x-mass commented Jul 10, 2024

Proof that CI passes if run with proper zkLLVM artifacts: https://github.com/NilFoundation/proof-producer/actions/runs/9871678319. The only diff is run-id to fetch artifacts from

Let's commit it as is and re-run check after zkLLVM is merged

@x-mass
Copy link
Member Author

x-mass commented Jul 10, 2024

Proof that zkLLVM CI using updated reusable worklflow passes as well:
https://github.com/NilFoundation/zkLLVM/actions/runs/9878173460

@x-mass x-mass merged commit f3e985d into master Jul 10, 2024
4 of 5 checks passed
@x-mass x-mass deleted the nix branch July 10, 2024 22:27
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