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

cargo: Update dependencies #414

Merged
merged 1 commit into from
Apr 26, 2024

Conversation

ErikDeSmedt
Copy link
Collaborator

Update all dependencies to the latest version.
We are lucky and don't have any conflicts

@ErikDeSmedt ErikDeSmedt force-pushed the cargo_update_dependencies branch from 15a60b1 to c780f73 Compare April 25, 2024 11:28
Randy808
Randy808 previously approved these changes Apr 25, 2024
@ErikDeSmedt ErikDeSmedt changed the title cargo: Update all dependencies cargo: Update dependencies Apr 25, 2024
@nepet
Copy link
Collaborator

nepet commented Apr 25, 2024

Did you also run backend checks? IIRC we had some trouble upgrading tokio in the past.

@ErikDeSmedt ErikDeSmedt force-pushed the cargo_update_dependencies branch from d452219 to c780f73 Compare April 25, 2024 15:11
@ErikDeSmedt
Copy link
Collaborator Author

Did you also run backend checks? IIRC we had some trouble upgrading tokio in the past.

Updating tokio was indeed broken.
I reverted the commit which update tonic and tokio in the gl-signer-proxy.

@ErikDeSmedt ErikDeSmedt marked this pull request as ready for review April 25, 2024 15:13
@cdecker
Copy link
Collaborator

cdecker commented Apr 25, 2024

ACK c780f73

@cdecker cdecker enabled auto-merge (rebase) April 25, 2024 15:20
I've used `cargo upgrade`.
@cdecker cdecker force-pushed the cargo_update_dependencies branch from c780f73 to 62eba8d Compare April 26, 2024 12:47
@cdecker cdecker merged commit 86c4364 into Blockstream:main Apr 26, 2024
17 checks passed
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.

4 participants