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

Reproducible Verification of Stylus Programs #43

Merged
merged 15 commits into from
Jul 1, 2024
Merged

Conversation

rauljordan
Copy link
Contributor

This PR carries out the work from #25 merged with the latest Stylus changes. It includes the ability to run every cargo stylus command in a Docker container by Rust version cargo stylus reproducible 1.77 deploy --private-key=$PRIV_KEY, and for verifying deployed Stylus programs against a local project with cargo stylus verify --deployment-tx=$DEPLOYMENT_TX_HASH

@@ -7,17 +7,26 @@ use crate::{
};
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Review this file

@@ -0,0 +1,71 @@
// Copyright 2023, Offchain Labs, Inc.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Review this file

@chrisco512 chrisco512 self-assigned this Jun 19, 2024
Copy link
Member

@joshuacolvin0 joshuacolvin0 left a comment

Choose a reason for hiding this comment

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

LGTM

rauljordan and others added 3 commits July 1, 2024 16:33
Co-authored-by: Joshua Colvin <jcolvin@offchainlabs.com>
This reverts commit 2e8d785.
@rauljordan
Copy link
Contributor Author

CI agent seems to be flawed. Will merge and check if other PRs also fail and address

@rauljordan rauljordan merged commit 5fac3da into main Jul 1, 2024
3 of 7 checks passed
@rauljordan rauljordan deleted the repro-verify-docker branch July 1, 2024 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants