-
Notifications
You must be signed in to change notification settings - Fork 56
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
Add reproducible builds #534
Conversation
@bobbinth followed the updates that you made on this PR: 0xPolygonMiden/crypto#296 and added them to this one. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, there are some files with extra unnecessary white lines that would be nice to clean up, but otherwise the PR looks good.
…t-toolchain" This reverts commit a159eb4.
[![test](https://github.com/0xPolygonMiden/base/actions/workflows/test.yml/badge.svg)](https://github.com/0xPolygonMiden/base/actions/workflows/test.yml) | ||
[![no-std](https://github.com/0xPolygonMiden/base/actions/workflows/no-std.yml/badge.svg)](https://github.com/0xPolygonMiden/base/actions/workflows/no-std.yml) | ||
[![RUST_VERSION](https://img.shields.io/badge/rustc-1.75+-lightgray.svg)]() | ||
[![CRATE](https://img.shields.io/crates/v/miden-base)](https://crates.io/crates/miden-base) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no miden-base
crate - so, this will show up as a broken link. Not sure what's the best way to handle this though. We could change this to show jus tmiden-objects
. Or show all 3 - miden-objects
, miden-lib
, and miden-tx
- but this may be too busy. Or we could get rid of it completely. Or maybe there is another way?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed here: #544
This PR improves the reliability of the builds for the miden-node enabling more consistency and reproducibility.
closes: #482