-
-
Notifications
You must be signed in to change notification settings - Fork 83
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
Added capability for .remote() to contact the DHT (BEP46) #38
Conversation
Add compiler requirements to travis config
@yciabaud fixed the travis issue, but now the |
poke @feross |
Sorry traveling in China for another week with sparse Internet. Will check
|
Any news on this? |
Just want to sub to this so I can get an update whenever @feross is around. |
@feross any news on this? The test errors are because of saucelabs credentials. It's missing the salt so I'll work on that as soon as I can. |
@feross @lmatteis is this just a matter of adding the following snippit at the beginning of test.js
I think it's related to this: defunctzombie/zuul#86 I also noticed that the travis link is broken. It points to feross/... but the actual location is webtorrent/... |
Well, a bit more than that. Basically copying what is done in webtorrent/package.json to short-circuit the sauce issue for PRs. I can submit a separate PR to do this if that's what's needed to unblock this PR. |
submitted #46 for what I mentioned above. Should be all squared away |
Have the CI issues been fixed since this was created? Would a new PR that adds this functionality be welcome? |
@DiegoRBaquero Was this closed because it's been implemented? Or is it just very out of date? |
Closed in favor of #58 |
Thank you for clarifying! |
Discussion here webtorrent/webtorrent#886 (comment)