LGTM! One nit: we may want to add a CI check to ensure `reqwest` is not accidentally added back to the `*-base` feature dependency graph in the future. Here is a suggestion: https://github.com/apache/arrow-rs-object-store/pull/724#discussion_r3399476082 _Originally posted by @kevinjqliu in https://github.com/apache/arrow-rs-object-store/pull/724#pullrequestreview-4480958253_
LGTM!
One nit: we may want to add a CI check to ensure
reqwestis not accidentally added back to the*-basefeature dependency graph in the future.Here is a suggestion: #724 (comment)
Originally posted by @kevinjqliu in #724 (review)