Publishers.Merge

[broadwaylamb]

• Implement the actual publishers.
• Don't forget about Publishers.MergeMany!
• Add tests.
• Resolve TODOs
• Can we use ManagedBuffer?
• Update documentation

[codecov]

Codecov Report

Merging #72 into master will decrease coverage by 7.33%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #72      +/-   ##
==========================================
- Coverage   98.22%   90.89%   -7.34%     
==========================================
  Files          83       72      -11     
  Lines        4562     3908     -654     
==========================================
- Hits         4481     3552     -929     
- Misses         81      356     +275

Impacted Files	Coverage Δ
...urces/OpenCombine/Helpers/Publishers._Merged.swift	0% <0%> (ø)
...ombine/Publishers/GENERATED-Publishers.Merge.swift	0% <0%> (ø)
Sources/OpenCombine/ObservableObject.swift	60% <0%> (-29.84%)	⬇️
...rces/OpenCombine/Publishers/Publishers.Print.swift	100% <0%> (ø)	⬆️
...s/OpenCombine/Subscribers/Subscribers.Demand.swift	100% <0%> (ø)	⬆️
...penCombine/Publishers/Publishers.Autoconnect.swift	100% <0%> (ø)	⬆️
...enCombine/Publishers/Publishers.IgnoreOutput.swift	100% <0%> (ø)	⬆️
...penCombine/Publishers/Publishers.SubscribeOn.swift	100% <0%> (ø)	⬆️
Sources/OpenCombine/Publishers/Record.swift	100% <0%> (ø)	⬆️
.../OpenCombine/Publishers/Publishers.ReceiveOn.swift	100% <0%> (ø)	⬆️
... and 35 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f861335...af0ae86. Read the comment docs.

[OpenCombineBot]

• 🚫 Limit vertical whitespace to a single empty line. Currently 2. (vertical_whitespace)

[ArthurChi]

Does it suspend?

[Peter-Schorn]

What's the status on this pull request? I need MergeMany for a library I'm creating. Is there any other way to merge the output of multiple publishers in the current version of this library?

[amirriyadh]

It's been 2 years since this issue was opened, is there any intent for you guys to work on it?

[MaxDesiatov]

There is intent, but none of the maintainers had free time to work on this issue. New pull requests that implement a corresponding test suite and make it pass are always welcome and I'd be happy to review them.

[Kyle-Ye]

For anyone interested on this topic:
I have added Merge implementation on my fork recently. See OpenSwiftUIProject#10

But it has some dependency here. (eg. #248 as I do not intend to support that old version of Swift)

If the upstream feels good about it and someone can help review it, I can open a pull request here.