Skip to content
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

fix: start and stop connection manager with libp2p #225

Merged
merged 1 commit into from
Jul 22, 2018

Conversation

jacobheun
Copy link
Contributor

Resolves #224

This will start the connection manager on libp2p.start and stop it on libp2p.stop. Previously users would need to start and stop it themselves.

@ghost ghost assigned jacobheun Jul 19, 2018
@ghost ghost added the status/in-progress In progress label Jul 19, 2018
@jacobheun jacobheun requested review from alanshaw and daviddias July 19, 2018 13:56
Copy link
Member

@daviddias daviddias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a test?

@jacobheun
Copy link
Contributor Author

@diasdavid added a test to verify subcomponents are being started and stopped along with libp2p.

@jacobheun
Copy link
Contributor Author

npm is having a bad day, I will check back tomorrow and re-run the tests when npm is feeling better.

test: add test to verify libp2p starts and stops the right things
test: add test for verifying disabled modules
fix: linting
@jacobheun
Copy link
Contributor Author

@diasdavid This should be good to merge now.

@daviddias daviddias merged commit 6106915 into master Jul 22, 2018
@daviddias daviddias deleted the fix/cm-autostart branch July 22, 2018 18:29
@ghost ghost removed the status/in-progress In progress label Jul 22, 2018
@daviddias daviddias mentioned this pull request Jul 22, 2018
14 tasks
maschad pushed a commit to maschad/js-libp2p that referenced this pull request Jun 21, 2023
…bp2p#225)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
maschad pushed a commit to maschad/js-libp2p that referenced this pull request Jun 21, 2023
## [6.0.1](libp2p/js-libp2p-tcp@v6.0.0...v6.0.1) (2022-11-16)

### Trivial Changes

* **deps-dev:** bump @libp2p/interface-mocks from 7.1.0 to 8.0.1 ([libp2p#225](libp2p/js-libp2p-tcp#225)) ([a271056](libp2p/js-libp2p-tcp@a271056))
maschad pushed a commit to maschad/js-libp2p that referenced this pull request Jun 21, 2023
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
maschad pushed a commit to maschad/js-libp2p that referenced this pull request Jun 21, 2023
## [7.0.4](libp2p/js-libp2p-mplex@v7.0.3...v7.0.4) (2022-11-23)

### Dependencies

* **dev:** bump it-map from 1.0.6 to 2.0.0 ([libp2p#225](libp2p/js-libp2p-mplex#225)) ([a153108](libp2p/js-libp2p-mplex@a153108))

### Trivial Changes

* update benchmark ([libp2p#232](libp2p/js-libp2p-mplex#232)) ([d73381e](libp2p/js-libp2p-mplex@d73381e))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants