-
Notifications
You must be signed in to change notification settings - Fork 231
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
Build 8.9.4 #320
Build 8.9.4 #320
Conversation
localtunnel/saucelabs is not really working atm. |
@mcollina FWIW memdown had similar issues, that disappeared when we switched to Sauce Connect (Level/memdown#81) - though it depends on an as-of-yet unmerged change to |
@vweevers considering that zuul is not really well maintained, how about we spin up a new project for that? I would prefer to depend on something from npm and not from a github account. |
SGTM! Can we reduce its scope (Sauce Connect only) to lower the maintenance cost? |
Yes definitely. |
Scaling CI down to just Sauce Connect sounds good! ~ have been frustrated with Zuul for a while too now; having a more reliable solution is definitely a step forward. |
I should have knocked on wood.. Sauce Labs is now also failing for memdown, with an error similar to #320 (comment). But I think forking zuul to work with Sauce Connect is still a good idea, as it at least removes localtunnel as a point of failure. If you agree I'll open an repo to continue discussion there. |
As titled.
I have also updated process-nextick-args.
Fixes #316.