Skip to content
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

Closed
wants to merge 1 commit into from

Conversation

blacktemplar
Copy link
Contributor

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

@blacktemplar blacktemplar added the ready-for-review The code is ready for review label Dec 12, 2020
@blacktemplar blacktemplar requested review from a user and AgeManning December 12, 2020 12:18
Copy link
Member

@AgeManning AgeManning left a 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?

@michaelsproul
Copy link
Member

This closes #2086, so I'm going to merge it as highest prio.

bors r+

bors bot pushed a commit that referenced this pull request Dec 14, 2020
## 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`
@michaelsproul michaelsproul added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Dec 14, 2020
@bors
Copy link

bors bot commented Dec 14, 2020

Build failed:

@michaelsproul
Copy link
Member

DHT test failed

bors retry

bors bot pushed a commit that referenced this pull request Dec 14, 2020
## 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`
@bors bors bot changed the title Update dependencies [Merged by Bors] - Update dependencies Dec 14, 2020
@bors bors bot closed this Dec 14, 2020
@blacktemplar
Copy link
Contributor Author

We don't need our hyper here because 13.8 has done the necessary updates for us right?

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge This PR is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants