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

Removing a pin to Sui version, using always latest instead #3

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

porkbrain
Copy link
Contributor

Because Suibase does not support pagination of Github releases, any pinned Sui version of testnet soon results in an error. We use the latest Sui tag instead to avoid this.

…inned Sui version of testnet soon results in an error. We use the latest Sui tag instead to avoid this.
@porkbrain porkbrain added the bug Something isn't working label Oct 10, 2024
@porkbrain porkbrain requested review from loverdos and tuky191 October 10, 2024 13:52
@porkbrain porkbrain self-assigned this Oct 10, 2024
@loverdos
Copy link
Contributor

Should we also update

Sui = { git = "https://github.com/MystenLabs/sui.git", subdir = "crates/sui-framework/packages/sui-framework", rev = "devnet-v1.27.0" }
to something more recent?

@porkbrain
Copy link
Contributor Author

Should we also update

Sui = { git = "https://github.com/MystenLabs/sui.git", subdir = "crates/sui-framework/packages/sui-framework", rev = "devnet-v1.27.0" }

to something more recent?

I believe I kept it on this version because that was the version of the protochain compiler. I wanted to make sure not to use any newer APIs. However, given that the contract is now "sealed", we could change the version. At the moment I don't see the benefit of doing that though, because we won't be at the same version as running examples anyway.

@pb-at-talus pb-at-talus merged commit 2d10a4c into main Oct 29, 2024
5 checks passed
@pb-at-talus pb-at-talus deleted the fix/use-latest-sui-release branch October 29, 2024 13:25
tuky191 pushed a commit that referenced this pull request Nov 6, 2024
Because Suibase does not support pagination of Github releases, any pinned Sui version of testnet soon results in an error. We use the latest Sui tag instead to avoid this.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants