Skip to content

Improve the kdoc on the newly-public collectToSink. #1262

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

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

zach-klippenstein
Copy link
Collaborator

No description provided.

@zach-klippenstein zach-klippenstein requested a review from a team as a code owner February 4, 2025 17:28
* to the [actionSink]. This operator propagates back pressure from the workflow runtime, so if there
* is a lot of contention on the workflow runtime the flow will be suspended while the action is
* queued.
* to the [actionSink]. This may be used as an alternative to a [asWorker] for certain [Flow]s.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* to the [actionSink]. This may be used as an alternative to a [asWorker] for certain [Flow]s.
* to the [actionSink]. This may be used as a lighter weight alternative to
* [asWorker] for certain [Flow]s.

* > **Warning:** Be careful using this from `runningSideEffect` when the source [Flow] comes from
* > state or props. The side effect will capture those values when it first runs, and if the instance
* > of the flow changes in a future render pass, the side effect will still be collecting the stale
* > [Flow]. In this case
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this case…?

@zach-klippenstein zach-klippenstein merged commit dc32e4c into main Feb 4, 2025
31 checks passed
@zach-klippenstein zach-klippenstein deleted the zachklipp/collecttosink-docs branch February 4, 2025 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants