build(deps): bump the reqwest group across 1 directory with 2 updates - #6773
build(deps): bump the reqwest group across 1 directory with 2 updates#6773dependabot[bot] wants to merge 1 commit into
Conversation
c5af032 to
06fcf91
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 06fcf9129b
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| ] } | ||
| reqwest-middleware = "0.4" | ||
| reqwest-retry = "0.8" | ||
| reqwest-middleware = "0.5" |
There was a problem hiding this comment.
Align reqwest with middleware 0.5
When quickwit-rest-client is compiled, Transport::new passes the workspace's reqwest::Client into reqwest_middleware::ClientBuilder::new, but the workspace still selects reqwest 0.12 while middleware 0.5 selects reqwest 0.13 (as shown by the two versions in Cargo.lock). Those client types come from distinct crate versions and are incompatible, so this crate—and therefore workspace builds including it—will not compile unless reqwest is upgraded alongside the middleware or these packages remain on compatible versions.
AGENTS.md reference: AGENTS.md:L121-L121
Useful? React with 👍 / 👎.
5b64a3a to
3388253
Compare
Bumps the reqwest group with 2 updates in the /quickwit directory: [reqwest-middleware](https://github.com/TrueLayer/reqwest-middleware) and [reqwest-retry](https://github.com/TrueLayer/reqwest-middleware). Updates `reqwest-middleware` from 0.4.2 to 0.5.2 - [Release notes](https://github.com/TrueLayer/reqwest-middleware/releases) - [Commits](TrueLayer/reqwest-middleware@reqwest-middleware-v0.4.2...reqwest-middleware-v0.5.2) Updates `reqwest-retry` from 0.8.0 to 0.9.1 - [Release notes](https://github.com/TrueLayer/reqwest-middleware/releases) - [Commits](https://github.com/TrueLayer/reqwest-middleware/commits/reqwest-retry-v0.9.1) --- updated-dependencies: - dependency-name: reqwest-middleware dependency-version: 0.5.2 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: reqwest - dependency-name: reqwest-retry dependency-version: 0.9.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: reqwest ... Signed-off-by: dependabot[bot] <support@github.com>
3388253 to
157559c
Compare
Bumps the reqwest group with 2 updates in the /quickwit directory: reqwest-middleware and reqwest-retry.
Updates
reqwest-middlewarefrom 0.4.2 to 0.5.2Release notes
Sourced from reqwest-middleware's releases.
Commits
614b947chore: release (#280)051dc25feat!: support opentelemetry 0.32 (#284)9bedaf8Adjust third-party middleware list in readme (#282)8bc3640feat(reqwest-middleware):streamfeature exported. (#246)de31517feat: expose internal reqwest::Client as reference (#209)0744f65chore: release (#274)33e6be9feat!: support the opentelemetry 0.31 (#272)d3773bbci: reduce debug-section size to get tests to run (#250)7171e65fix(reqwest-retry): drop instant by upgrading wasmtimer (#254)1692cadSet changelog version for last release (#268)Updates
reqwest-retryfrom 0.8.0 to 0.9.1Release notes
Sourced from reqwest-retry's releases.
Commits