-
Notifications
You must be signed in to change notification settings - Fork 811
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
[Merged by Bors] - Update dependencies #2084
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't need our hyper here because 13.8 has done the necessary updates for us right?
This closes #2086, so I'm going to merge it as highest prio. bors r+ |
## Issue Addressed Partially addresses dependencies mentioned in issue #1712. ## Proposed Changes Updates dependencies (including an update avoiding a vulnerability) + add tokio compatibility to `remote_signer_test`
Build failed: |
DHT test failed bors retry |
## Issue Addressed Partially addresses dependencies mentioned in issue #1712. ## Proposed Changes Updates dependencies (including an update avoiding a vulnerability) + add tokio compatibility to `remote_signer_test`
Pull request successfully merged into unstable. Build succeeded: |
I don't think it was needed at all. The remote signer was the only crate using the fork and all the others already use upstream. At least it compiles and all tests run through, but this is the reason why I requested a review from @hermanjunge. Maybe he knows more why the fork dependency was needed. |
Issue Addressed
Partially addresses dependencies mentioned in issue #1712.
Proposed Changes
Updates dependencies (including an update avoiding a vulnerability) + add tokio compatibility to
remote_signer_test