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

Bump version to 21.2.1 #1445

Closed
wants to merge 4 commits into from
Closed

Bump version to 21.2.1 #1445

wants to merge 4 commits into from

Conversation

leighmcculloch
Copy link
Member

@leighmcculloch leighmcculloch commented Aug 19, 2024

What

Bump version to 21.2.1, creating release branch.

Why

Created by @leighmcculloch manually because the Bump Version workflow is currently erroring with permission errors. Related links about those failures:

What is next

See the release instructions for a full rundown on the release process:
https://github.com/stellar/actions/blob/main/README-rust-release.md

Commit any changes to the release/v21.2.1 branch that are needed in this release.

If this is a regular release releasing from main, merge this PR when ready, and after merging, create a release for this version by going to this link: https://github.com/stellar/rs-soroban-env/releases/new?tag=v21.2.1&title=21.2.1

If this is a backport or patch release of a past version, see the release instructions. When ready to release this branch create a release by going to this link:
https://github.com/stellar/rs-soroban-env/releases/new?tag=v21.2.1&title=21.2.1&target=release/v21.2.1

@leighmcculloch leighmcculloch requested a review from a team as a code owner August 19, 2024 10:28
@dmkozh
Copy link
Contributor

dmkozh commented Aug 19, 2024

leighmcculloch wants to merge 2 commits into main

This should go into a separate branch, not into main, right?

@leighmcculloch
Copy link
Member Author

leighmcculloch wants to merge 2 commits into main

This should go into a separate branch, not into main, right?

I'll close the PR without merging it once tagged/released. We don't need the change to go to any branch, it's just a visible place to track and open piece of work.

### What
Unpin all non-stellar dependencies.

### Why
We have had reports that the libs are difficult to integrate into other
projects due to the rigid dependency requirements.

We ultimately need the dependencies to be pinned in stellar-core.

There's benefit to the tests in this repo running with the same
dependencies as what core pins to.

There's benefits to stellar-rpc also using the same dependencies so as
to decrease the chance that simulation runs with different behavior.
Also, the same applies to the SDK for test behavior.

Pinning the deps were an easy way to keep the dependencies consistent
everywhere, but it also makes it difficult for folks to use the env
libraries.

Close #1351

### Other things that need to happen

@graydon is making a tool to check that Cargo.lock files are using
consistent versions where possible across the env repo, core repo, and
we can also use that in the rpc repo. It won't be practical to expect
contract devs to use it on their contract projects, so contract devs
will be able to use whatever combination of dependencies and that's just
a limitation.

### Why not

We could say again that not doing this has greater benefits than
allowing folks to more easily use the libraries in their own projects.

(cherry picked from commit a54d997)
@leighmcculloch leighmcculloch marked this pull request as draft August 20, 2024 13:06
@leighmcculloch leighmcculloch marked this pull request as ready for review August 20, 2024 13:06
@leighmcculloch
Copy link
Member Author

@leighmcculloch leighmcculloch deleted the release/v21.2.1 branch August 20, 2024 20: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.

2 participants