Skip to content
This repository has been archived by the owner on Jul 21, 2023. It is now read-only.

fix: find providers should yield when found locally #160

Merged

Conversation

vasco-santos
Copy link
Member

If the locally known providers were enough to achieve the maxNumProviders option, they were not being yield, but returned

@vasco-santos vasco-santos force-pushed the fix/find-providers-should-yield-when-found-locally branch from 0b74738 to 1439f67 Compare December 5, 2019 10:26
Copy link
Contributor

@jacobheun jacobheun left a comment

Choose a reason for hiding this comment

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

Looks good!

@vasco-santos vasco-santos merged commit e40834a into master Dec 5, 2019
@vasco-santos vasco-santos deleted the fix/find-providers-should-yield-when-found-locally branch December 5, 2019 12:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants