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

fix: upgrade to axum 0.7 #609

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

fix: upgrade to axum 0.7 #609

wants to merge 10 commits into from

Conversation

chris13524
Copy link
Member

@chris13524 chris13524 commented Apr 5, 2024

Description

Updates to latest version of utils-rs and thus axum 0.7 and http/hyper v1 deps.

Also:

  • Removes unused src/extractors which were giving some clippy warnings.

Remaining work:

How Has This Been Tested?

Existing tests?

Due Diligence

  • Breaking change
  • Requires a documentation update
  • Requires a e2e/integration test update

@chris13524 chris13524 self-assigned this Apr 5, 2024
@geekbrother
Copy link
Contributor

Nice 🙌 I added this to my backlog as well to update the Axum to the latest. Thanks for doing this 🙏

@chris13524 chris13524 marked this pull request as ready for review April 8, 2024 13:24
@Goonte
Copy link

Goonte commented May 8, 2024

Thanks buddy I just wanted to know if I made a deposit is the contract going to take my fees for gas funds why I'm asking I keep adding but I don't receive any back to use.

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