Change Discover to be a sealed trait #443
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Discover
was really just aTryStream<Item = Change>
, so thischange makes that much clearer. Specifically, users are intended to use
Discover
only in bounds, whereas implementors should implementStream
with the appropriateItem
type.Discover
then comes with ablanket implementation for anything that implements
TryStream
appropriately. This obviates the need for the
discover::stream
module.