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

Update CLI and soroban-rpc to the latest XDR changes #846

Merged
merged 23 commits into from
Aug 10, 2023

Conversation

2opremio
Copy link
Contributor

@2opremio 2opremio commented Aug 7, 2023

What

Update CLI and soroban-rpc to the latest XDR changes

Why

Fixes part of #844

Known limitations

N/A

@willemneal
Copy link
Member

Should this PR target main? Shouldn't main stay in line with futurenet's XDR?

cmd/soroban-rpc/lib/preflight/src/fees.rs Outdated Show resolved Hide resolved
cmd/soroban-rpc/lib/preflight/src/fees.rs Outdated Show resolved Hide resolved
cmd/soroban-rpc/lib/preflight/src/fees.rs Outdated Show resolved Hide resolved
cmd/soroban-rpc/lib/preflight/src/fees.rs Outdated Show resolved Hide resolved
cmd/soroban-rpc/lib/preflight/src/fees.rs Outdated Show resolved Hide resolved
cmd/soroban-rpc/lib/preflight/src/fees.rs Outdated Show resolved Hide resolved
cmd/soroban-rpc/lib/preflight/src/fees.rs Show resolved Hide resolved
cmd/soroban-rpc/lib/preflight/src/fees.rs Outdated Show resolved Hide resolved
cmd/soroban-rpc/lib/preflight/src/fees.rs Outdated Show resolved Hide resolved
@tsachiherman
Copy link
Contributor

Should this PR target main? Shouldn't main stay in line with futurenet's XDR?

I believe that it's already not aligned with futurenet, but rather an intermediate xdr.

@2opremio
Copy link
Contributor Author

2opremio commented Aug 8, 2023

Should this PR target main? Shouldn't main stay in line with futurenet's XDR?

Main already contains XDR changes ahead of futurenet. The futurenet code was tagged at https://github.com/stellar/soroban-tools/tree/v0.9.5

@willemneal
Copy link
Member

Ah cool good to know! So we can't release the CLI again until futurenet is aligned?

Copy link
Contributor

@Shaptic Shaptic left a comment

Choose a reason for hiding this comment

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

not qualified to approve this lol but adding some thoughts from an outside perspective

@Shaptic
Copy link
Contributor

Shaptic commented Aug 8, 2023

@willemneal we could branch off of v0.9.5 and cherry-pick, but I don't know if our CI/CD supports publishing crates with that workflow?

Copy link
Contributor

@tamirms tamirms left a comment

Choose a reason for hiding this comment

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

I'm not able to review the preflight lib changes but everything else looks good

@2opremio 2opremio enabled auto-merge (squash) August 10, 2023 10:14
@2opremio 2opremio merged commit 438656a into stellar:main Aug 10, 2023
@2opremio 2opremio deleted the bump-xdr branch August 10, 2023 12:11
paulbellamy pushed a commit that referenced this pull request Aug 18, 2023
paulbellamy pushed a commit that referenced this pull request Aug 18, 2023
paulbellamy pushed a commit that referenced this pull request Aug 21, 2023
* Fix ledgerentry visibility bug from #846

* Fix test output
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.

7 participants