search

passabilities / crypto-exchange

Pulls together list of crypto exchanges to interact with their API's in a uniform fashion.
MIT License
261 stars 78 forks source link

yard add fail #13

Closed lookfirst closed 7 years ago

lookfirst commented 7 years ago 
yarn add crypto-exchange
yarn add v0.27.5
[1/4] Resolving packages...
warning crypto-exchange > bitfinex > request > tough-cookie@0.9.15: ReDoS vulnerability parsing Set-Cookie https://nodesecurity.io/advisories/130
warning crypto-exchange > bitfinex > request > node-uuid@1.4.8: Use uuid module instead
warning crypto-exchange > coinbase > request > node-uuid@1.4.8: Use uuid module instead
warning crypto-exchange > yunbi-api-module > crypto@0.0.3: This package is no longer supported. It's now a built-in Node module. If you've depended on crypto, you should switch to the one that's built-in.
error Couldn't find match for "cb-accounts" in "felix-cleanup,felix-es6ify,master,plorio-update_validation,update_readme,version-0.3.2,withdraw-crypto,v0.2.2,v0.3.0,v0.3.1,v0.3.2,v0.3.3-0,v0.4.2,v0.4.3" for "https://github.com/passabilities/gdax-node.git".
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.
lookfirst commented 7 years ago

To be fair, npm fails too. ;-)

npm install crypto-exchange
npm ERR! code 1
npm ERR! Command failed: /usr/local/bin/git checkout cb-accounts
npm ERR! error: pathspec 'cb-accounts' did not match any file(s) known to git.
npm ERR!
passabilities commented 7 years ago

Problem solved: https://github.com/coinbase/gdax-node/pull/89#issuecomment-320651352

lookfirst commented 7 years ago

How is it solved?

passabilities commented 7 years ago

sorry about that, I thought I changed the reference of gdax-node and assumed that my change was added to npm after it was merged.

I repushed the cb-accounts branch