interledgerjs / settlement-lightning

⚡️ Settle Interledger payments using the Lightning Network
Apache License 2.0
15 stars 4 forks source link

fix: do not peer on connect #32

Closed karzak closed 5 years ago

karzak commented 5 years ago

fixes https://github.com/interledgerjs/ilp-plugin-lightning/issues/31

In the future, if we want to support opening of channels between servers/clients we can re-implement peering, although it shouldn't happen automatically on connect.

codecov-io commented 5 years ago

Codecov Report

Merging #32 into master will decrease coverage by 0.64%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
- Coverage   83.88%   83.23%   -0.65%     
==========================================
  Files           6        6              
  Lines         335      334       -1     
  Branches       17       17              
==========================================
- Hits          281      278       -3     
- Misses         50       52       +2     
  Partials        4        4
Impacted Files Coverage Δ
src/account.ts 78.29% <100%> (-0.17%) :arrow_down:
src/index.ts 90.1% <100%> (ø) :arrow_up:
src/lightning.ts 77.77% <0%> (-3.18%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 9efe188...053ed07. Read the comment docs.