Skip to content
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

Replace Peekable in PowerDistributor with Receivers #772

Closed
shsms opened this issue Nov 6, 2023 · 1 comment
Closed

Replace Peekable in PowerDistributor with Receivers #772

shsms opened this issue Nov 6, 2023 · 1 comment
Assignees
Labels
part:❓ We need to figure out which part is affected priority:❓ We need to figure out how soon this should be addressed type:enhancement New feature or enhancement visitble to users
Milestone

Comments

@shsms
Copy link
Contributor

shsms commented Nov 6, 2023

What's needed?

The PowerDistributor is the only place in the SDK where Peekables are used. Peekable is a confusing interface and should be avoided.

Proposed solution

No response

Use cases

No response

Alternatives and workarounds

No response

Additional context

No response

@shsms shsms added part:❓ We need to figure out which part is affected priority:❓ We need to figure out how soon this should be addressed type:enhancement New feature or enhancement visitble to users labels Nov 6, 2023
@christianparpart christianparpart self-assigned this Nov 6, 2023
@shsms shsms assigned shsms and unassigned christianparpart Nov 16, 2023
@llucax llucax added this to the v1.0.0-rc4 milestone Jan 15, 2024
@shsms
Copy link
Contributor Author

shsms commented Jan 19, 2024

Done here: #800

@shsms shsms closed this as completed Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part:❓ We need to figure out which part is affected priority:❓ We need to figure out how soon this should be addressed type:enhancement New feature or enhancement visitble to users
Projects
Development

No branches or pull requests

3 participants