Skip to content
This repository has been archived by the owner on Sep 21, 2024. It is now read-only.

chore: Pin reqwest to 0.11.20 to avoid issues with #686. #688

Merged
merged 2 commits into from
Nov 13, 2023
Merged

chore: Pin reqwest to 0.11.20 to avoid issues with #686. #688

merged 2 commits into from
Nov 13, 2023

Conversation

jsantell
Copy link
Contributor

No description provided.

@jsantell jsantell requested a review from cdata October 19, 2023 15:54
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 31, 2023 •
edited
Loading

Test flake analysis

No flakes detected 🎉

status platform features toolchain
🟢 macos-13 test-kubo,headers stable
🟢 ubuntu-latest test-kubo,headers,rocksdb stable
🟢 ubuntu-latest test-kubo,headers stable
🟢 windows-latest test-kubo,headers stable
🟢 ubuntu-latest test-kubo,headers nightly
🟢 macos-13 test-kubo,headers,rocksdb stable

@cdata
Copy link
Collaborator

cdata commented Nov 1, 2023

To pin, you need to use the = prefix. Otherwise, the default resolution for a version is: 1.2.3 := >=1.2.3, <2.0.0

https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#specifying-dependencies-from-cratesio

@jsantell
Copy link
Contributor Author

jsantell commented Nov 1, 2023

To pin, you need to use the = prefix. Otherwise, the default resolution for a version is: 1.2.3 := >=1.2.3, <2.0.0

https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#specifying-dependencies-from-cratesio

good catch!

cdata
cdata previously approved these changes Nov 13, 2023
View reviewed changes
Copy link
Collaborator

@cdata cdata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cdata cdata merged commit e3ab34a into main Nov 13, 2023
@cdata cdata deleted the pin-reqwest branch November 13, 2023 21:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cdata cdata cdata approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jsantell @cdata