-
Notifications
You must be signed in to change notification settings - Fork 48
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
Bump flow-go-sdk
& cadence
versions
#779
Bump flow-go-sdk
& cadence
versions
#779
Conversation
which ones? they may have been not implemented. |
@bluesign I know that they are not yet implemented by Emulator, I just need an Emulator version that uses |
Thank you for already starting to update the Cadence dependency! This probably also needs to update flow-go. We have a tool in the Cadence repo which automatically updates all necessary dependencies at https://github.com/onflow/cadence/tree/master/tools/update. We can probably merge this PR for now, but might want to do a follow-up |
That's a good point, but I am not sure as to which tag to use. The current is |
flow-go is an exception in the typical tagging scheme of all repos in the org: it is only tagged for network updates, which happen very infrequently. So normally we use a latest commit of it when updating the downstream dependencies (i.e., all the developer tools) |
Oh, you're totally right 😅 I completely forgot that I could do something like: go get github.com/onflow/flow-go@92474d3 @turbolent Updated the |
Is there anything blocking us from merging this? I could really use a new emulator release with the latest Cadence version |
@joshuahannan Should be good to go, though I don't have merge rights 😅 |
Relevant issue: onflow/flow-evm-gateway#727
Relevant PR: onflow/flow-evm-gateway#728
Description
Needed for some new subscription methods available on
flow-go-sdk
.Also updates the breaking changes coming from the version bump on
cadence
, related to debugger.For contributor use:
master
branchFiles changed
in the GitHub PR explorer