Replace `Merge` and `MergeNamed` with `merge()`

frequenz-floss / frequenz-channels-python

Channel implementations for Python

https://frequenz-floss.github.io/frequenz-channels-python/

MIT License

7 stars 8 forks source link

Replace `Merge` and `MergeNamed` with `merge()` #238

Closed llucax closed 10 months ago

llucax commented 11 months ago

Remove MergeNamed
Add a merge() wrapper function that replaces Merge
Rename Merge to _Merge to make it private more explicitly
Rename _Merge variable args to receivers
Improve string representation of merge receiver
Raise an exception is merge() is called with less than two receivers

Fixes #236, fixes #237.

llucax commented 10 months ago

Updated. Sorry @shsms, I forgot to create fixup/new commits for all the changes and amended one one the old commits (to only raise when no receivers are passed). The diff for that amend is this one.

All the other changes are in new commits.

shsms commented 10 months ago

Thanks @llucax, I have no further comments, will approve once you fixup.

llucax commented 10 months ago

Rebased to apply the fixups and enabled auto-merge.