Skip to content

Conversation

@guillaumemichel
Copy link
Collaborator

@guillaumemichel guillaumemichel commented Jul 24, 2025

Part of #1095

Depends on #1114


Network operations performed by the provider.

@guillaumemichel guillaumemichel requested a review from a team as a code owner July 24, 2025 10:10
This was referenced Jul 24, 2025
Copy link
Contributor

@gammazero gammazero left a comment

Choose a reason for hiding this comment

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

Please simplify code by using package github.com/ipfs/go-test/random
After that, ok to merge

@guillaumemichel guillaumemichel changed the base branch from provider-queues to provider August 5, 2025 11:20
@guillaumemichel guillaumemichel changed the base branch from provider to provider-queues August 5, 2025 11:21
@guillaumemichel guillaumemichel changed the base branch from provider-queues to provider August 5, 2025 11:21
@guillaumemichel guillaumemichel merged commit d8c733d into provider Aug 5, 2025
9 checks passed
guillaumemichel added a commit that referenced this pull request Aug 19, 2025
* provider: adding provide and reprovide queue

* provider: network operations

* add some tests

* addressed review

* use go-test/random

* satisfy linter
guillaumemichel added a commit to guillaumemichel/go-libp2p-kad-dht that referenced this pull request Sep 17, 2025
* provider: adding provide and reprovide queue

* provider: network operations

* add some tests

* addressed review

* use go-test/random

* satisfy linter
guillaumemichel added a commit that referenced this pull request Sep 17, 2025
* provider: adding provide and reprovide queue

* provider: network operations

* add some tests

* addressed review

* use go-test/random

* satisfy linter
guillaumemichel added a commit that referenced this pull request Sep 18, 2025
* provider: adding provide and reprovide queue

* provider: network operations

* add some tests

* addressed review

* use go-test/random

* satisfy linter
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.

3 participants