uber-node / tcurl

A command line utility to talk to tchannel servers
MIT License
36 stars 7 forks source link

Disable tracing #134

Closed isaachier closed 6 years ago

CLAassistant commented 6 years ago

CLA assistant check
All committers have signed the CLA.

kriskowal commented 6 years ago

Like most of the world, npm has abandoned our CI which runs in Node.js 0.10. To get test passing, we will have to disable CI for that environment. This is a bit of a conundrum since tcurl is such a deeply legacy tool, the only relevant testing environment is very old versions of Node.js. It’s also unlikely that tcurl is installable anymore on those environments.

I suppose I’d be fine with migrating the CI forward in .travis.yaml to test only Node.js LTS and LTS-1.

kriskowal commented 6 years ago

Poke me if you get the tests passing and land this. I’ll cut the release.

isaachier commented 6 years ago

Sorry for the delay. Thanks @kriskowal, really appreciate the help here.

isaachier commented 6 years ago

Seems this finally works @kriskowal.

kriskowal commented 6 years ago

Published.