bitcore-p2p icon indicating copy to clipboard operation
bitcore-p2p copied to clipboard

Use bitcore-lib as a peerDependency

Open CaptEmulation opened this issue 9 years ago • 5 comments

One wrinkle to using the bitcore stack is the obvious need to have a singleton bitcore-lib. The bitcoin-lib package hosts the bitcoind library and there are good reasons to not wanting to have multiple bitcoind instances running on the same machine.

However, I am not a fan of the runtime check and would rather have all packages be declaring bitcoin-lib in peerDependencies. This way the enclosing package can specify whatever version of bitcoin-lib it wants and npm will inform user if there is an unmet dependency at install time rather than run time.

I have a working example of bitcore-lib running as a peerDependency at bitcore-docker-build which is also running testnet at bitcore.soapbubble.online/insight

I also updated bitcore-lib to 0.14 because bitcore-wallet-service has already done so and I figured if it was good enough for bitcoin-wallet-service it's good enough for everything else :smile:

CaptEmulation avatar Mar 25 '17 18:03 CaptEmulation

Also depends on:

  • https://github.com/bitpay/bitcore-node/pull/500
  • https://github.com/bitpay/insight-api/pull/501
  • https://github.com/bitpay/bitcore-wallet-service/pull/639
  • https://github.com/bitpay/bitcore-message/pull/27

CaptEmulation avatar Mar 25 '17 18:03 CaptEmulation

Coverage Status

Coverage remained the same at 100.0% when pulling 42122cb08a4b8e47a87419369adf892dd09974f5 on CaptEmulation:bitcore-lib-as-peer-dependency into b7a6845a4f4c7940884247113a208fe30fcc0e39 on bitpay:master.

coveralls avatar Mar 25 '17 18:03 coveralls

Coverage Status

Coverage remained the same at 100.0% when pulling 982c9c43e4c6769917e1f1d443c794f3511e2792 on CaptEmulation:bitcore-lib-as-peer-dependency into b7a6845a4f4c7940884247113a208fe30fcc0e39 on bitpay:master.

coveralls avatar Mar 25 '17 18:03 coveralls

Coverage Status

Coverage remained the same at 100.0% when pulling be7c3d1f197b7ecc5b304b869e005c5a660bd758 on CaptEmulation:bitcore-lib-as-peer-dependency into b7a6845a4f4c7940884247113a208fe30fcc0e39 on bitpay:master.

coveralls avatar Mar 25 '17 22:03 coveralls

Coverage Status

Coverage remained the same at 100.0% when pulling 1bb691981dd5c3c0746c75353683f5b2f03a9496 on CaptEmulation:bitcore-lib-as-peer-dependency into b7a6845a4f4c7940884247113a208fe30fcc0e39 on bitpay:master.

coveralls avatar Mar 25 '17 22:03 coveralls