New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deprecate succeedLazy #1377
Deprecate succeedLazy #1377
Conversation
93e1e2e
to
8c696dd
Compare
8c696dd
to
1021af8
Compare
7aa40ef
to
9d27e5d
Compare
9d27e5d
to
e18cb07
Compare
03f88bb
to
96e92f8
Compare
849ffd9
to
dd24d0d
Compare
dd24d0d
to
c1792c1
Compare
@shankarshastri Fixed docs. |
I think we should delegate to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the hard work! 🙏 Some comments, and I do think the deprecated methods should redirect to effectTotal
(or possibly stay unmodified) to not break callers on upgrade.
What is the impact of this on |
|
4806bcf
to
9836ca5
Compare
9836ca5
to
ba400c7
Compare
ba400c7
to
72c2462
Compare
@ioleo Excellent work, thank you for this! 🙏 |
Fixes #1346
succeedLazy
methods.succeed
.succeed
toZSink
,Sink
andZStreamChunk
.succeedLazy
test fromRTSSpec
succeedLazy
witheffect
oreffectTotal
where laziness was essential and withsucceed
where lazyness was not required