Skip to content
This repository has been archived by the owner on May 26, 2022. It is now read-only.

More consistent use of options #25

Merged
merged 3 commits into from
May 17, 2019
Merged

More consistent use of options #25

merged 3 commits into from
May 17, 2019

Conversation

vyzo
Copy link
Contributor

@vyzo vyzo commented May 16, 2019

  • The routing-backed discovery implementation respects the TTL option in Advertise
  • The utility functions Advertise and FindPeers accept options

@vyzo vyzo requested a review from raulk May 16, 2019 15:34
@whyrusleeping whyrusleeping added the status/in-progress In progress label May 16, 2019
@ghost ghost assigned vyzo May 16, 2019
@vyzo vyzo requested a review from Stebalien May 17, 2019 08:25
@vyzo vyzo merged commit 74aa1c8 into master May 17, 2019
@vyzo vyzo deleted the feat/options branch May 17, 2019 19:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Stebalien Stebalien Stebalien approved these changes

@raulk raulk Awaiting requested review from raulk

Assignees

@vyzo vyzo

Labels
status/in-progress In progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vyzo @Stebalien @whyrusleeping