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

Check in compiled contracts required for build #73

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

jkilpatr
Copy link
Collaborator

This is unfortunately required for a reliable build experience. We've done this before, gone back on this decision due to the lack of transparency when checking in any binary output and then come back here out of pragmatisim when it comes to the difficulty of actually executing builds and ensuring those making their own builds don't have issues.

This contract bytecode is fully reproducbile by anyone else building it, and explorer verification should also cross verify.

This is unfortunately required for a reliable build experience. We've
done this before, gone back on this decision due to the lack of
transparency when checking in any binary output and then come back here
out of pragmatisim when it comes to the difficulty of actually executing
builds and ensuring those making their own builds don't have issues.

This contract bytecode is fully reproducbile by anyone else building
it, and explorer verification should also cross verify.
@jkilpatr jkilpatr force-pushed the jkilpatr/check-in-contracts branch from 8d043ec to b8eda96 Compare January 13, 2025 02:33
Copy link
Collaborator

@ChristianBorst ChristianBorst left a comment

Choose a reason for hiding this comment

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

Looks good to me, locally verified contract bin contents.

@jkilpatr jkilpatr merged commit b8eda96 into main Jan 13, 2025
12 of 15 checks passed
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