Skip to content

Conversation

@AbrarNitk
Copy link
Contributor

Description

In this PR we have upgraded the bb8 to 0.9 and arango-rs to 0.6, because we are also using recent version of bb8-redis which uses the bb8 version 0.9, so different versions are conflicting bb8.

Supporting information

Link to other information about the change, such as, GitHub issues or documentation.

Dependencies

  • List of the dependencies for this change, otherwise write "N/A"

Testing instructions

Please provide detailed step-by-step instructions for testing this change.

Other information

Include other information, such as, which sources and targets are affected.

@AbrarNitk AbrarNitk changed the title Bump arango-rs and bb8 deps version Bump arango-rs(0.5 to 0.6) and bb8(0.8 to 0.9) deps version Oct 23, 2025
@gabor-boros
Copy link
Owner

Hey @AbrarNitk, thank you for the PR! I thought nobody else uses this crate. Great to see that I was wrong.

@gabor-boros gabor-boros merged commit 9d11efc into gabor-boros:main Oct 24, 2025
1 check passed
@gabor-boros
Copy link
Owner

Although it is merged, just to double check: What was the reason behind changing use arangors::uclient::reqwest::ReqwestClient; to use arangors::client::reqwest::ReqwestClient; and removing #[async_trait]?

@gabor-boros
Copy link
Owner

@AbrarNitk do you have any insights?

@AbrarNitk
Copy link
Contributor Author

AbrarNitk commented Nov 4, 2025

Hey @gabor-boros, In arango dependency module name has changed from uclient to client. We do not require async_trait in the newer version of rust, now we can have async functions in traits since 1.75.

https://blog.rust-lang.org/2023/12/21/async-fn-rpit-in-traits/

@gabor-boros
Copy link
Owner

Thanks @AbrarNitk! I'm going to publish the new version soon.

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