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 the main branch of the risc0 monorepo for development #93

Merged
merged 42 commits into from
May 15, 2024

Conversation

nategraf
Copy link
Contributor

@nategraf nategraf commented Apr 30, 2024

This changes all the references for packages from risc0 in the Cargo.toml files. During development, we want to stay up to date with the main branch of risc0, and only switch to a stable release during the release process. With 1.0 approaching very soon now, the rate of breaking changes should slow significantly.

Resolves: #73
Resolves: #89

Co-authored-by: carlosgj94

@nategraf nategraf requested a review from capossele April 30, 2024 22:35
@nategraf
Copy link
Contributor Author

This PR sits on top of #87

Base automatically changed from victor/release-process-update-apr25 to main April 30, 2024 23:12
@nategraf
Copy link
Contributor Author

nategraf commented May 1, 2024

CI is failing because it installs cargo-risczero 0.21 instead of from main. I will work on fixing that.

@nategraf
Copy link
Contributor Author

nategraf commented May 2, 2024

Will have to think how effects the release, in particular, testing the release branch.

@nategraf
Copy link
Contributor Author

nategraf commented May 8, 2024

Getting closer! It looks like all the tests running with either cuda or metal fail at test bonsai_rest_api_mock::test::local_bonsai. Error is not particularly helpful and no idea off the top of my head why this might be happening. I can keep looking into this later.

@capossele capossele self-assigned this May 14, 2024
@nategraf
Copy link
Contributor Author

Ci failures could be a caching issue. Should try resetting the cache. Example
4f127ca

@nategraf nategraf merged commit 5959f6c into main May 15, 2024
8 of 9 checks passed
@nategraf nategraf deleted the victor/use-risc0-main-on-main branch May 15, 2024 17:18
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.

Update alloy and revm dependencies Update risc0-ethereum main branch to point to risc0 main branch
2 participants