Skip to content
This repository has been archived by the owner on Oct 14, 2022. It is now read-only.

Dependency Updates #86

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

martinheidegger
Copy link
Contributor

@martinheidegger martinheidegger commented Feb 13, 2018

Several of the dependencies of hashbase have been updated. In this PR I updated the dependencies that were easy to update. I didn't update leveldb and nodemailer because those dependencies would involve (significantly) more effort.

@martinheidegger
Copy link
Contributor Author

Note: The tests pass through on my machine but occasionally they fail. Maybe run again on travis can fix them?

@pfrazee
Copy link
Member

pfrazee commented Feb 13, 2018

Something is consistently failing with logins. I haven't been able to isolate it but it looks like races -- maybe something got screwed up with promises?

@martinheidegger
Copy link
Contributor Author

@pfrazee the tests run smooth here. would you mind to triangulate which commit broke it?

@pfrazee
Copy link
Member

pfrazee commented Feb 15, 2018

Sorry Im a bit tied up atm, but this is on my todo list

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants