TopicDefinition<IEnumerable<...>> should be TopicDefinition<IReadOnlyCollection<...>>
This is because we want to make sure that the variables passed in aren't being evaluated "lazily". By swapping to a collection, it's ensured that we're doing the looping eagerly
We should make some docs outlining this pattern as well
Noggog
commented
2 months ago
TopicDefinition<IEnumerable<...>>should beTopicDefinition<IReadOnlyCollection<...>>This is because we want to make sure that the variables passed in aren't being evaluated "lazily". By swapping to a collection, it's ensured that we're doing the looping eagerly
We should make some docs outlining this pattern as well