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

Replace zuul with airtap #1309

Merged
merged 6 commits into from Feb 21, 2018
Merged

Replace zuul with airtap #1309

merged 6 commits into from Feb 21, 2018

Conversation

@feross
Copy link
Member

feross commented Feb 21, 2018

Testing out a replacement package for zuul called airtap.

https://www.npmjs.com/package/airtap

Much simpler than zuul in that it only supports tap/tape, actively maintained, doesn't rely on the unreliable localtunnel service.

It's still super early software, so I want to test it on a few repos first. Currently added it to these repos:

  • simple-peer
  • magnet-uri
  • buffer
  • webtorrent (this PR)

It may have big breaking changes before V1, but it's already so much more reliable that I think it's worth it.

feross added 6 commits Feb 21, 2018
It's worth taking a bit of extra time to avoid npm cache bugs, and to get a more accurate test of a fresh install on a new user's machine
Not ideal to lose Electron testing, but I think this is net win for now.
@feross

This comment has been minimized.

Copy link
Member Author

feross commented Feb 21, 2018

Looks good!

@feross feross merged commit 2a998b7 into master Feb 21, 2018
5 checks passed
5 checks passed
Node Security No known vulnerabilities found
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@feross feross deleted the airtap branch Feb 21, 2018
@lock lock bot locked as resolved and limited conversation to collaborators May 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.