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.

Sign up for GitHub

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

Jump to bottom

rx: All operator #152

Closed
Tracked by #122
plastikfan opened this issue Apr 8, 2024 · 1 comment · Fixed by #153
Closed
Tracked by #122

rx: All operator #152

plastikfan opened this issue Apr 8, 2024 · 1 comment · Fixed by #153
Assignees
@plastikfan
Labels
feature New feature or request

Comments

@plastikfan
Copy link
Contributor

plastikfan commented Apr 8, 2024 •
edited
Loading

The predicate needs to be modified to accept Item[T], not T:

Predicate[T any] func(T) bool

This is to enable the correct handling of the value, as now we have a new auxilliary field on Item, B, which represents a boolean value, required by the All operator.
@plastikfan plastikfan mentioned this issue Apr 8, 2024
Open
59 tasks
@plastikfan plastikfan added the feature New feature or request label Apr 8, 2024
@plastikfan plastikfan self-assigned this Apr 8, 2024
@plastikfan plastikfan changed the title All rx: All operator Apr 8, 2024
@plastikfan
Copy link
Contributor Author

plastikfan commented Apr 8, 2024 •
edited
Loading

Parallel tests added but disabled because WithCPUPool is not full implemented yet.

plastikfan added a commit that referenced this issue Apr 8, 2024
@plastikfan
feat(rx): add all operator (#152)
4c89402
plastikfan added a commit that referenced this issue Apr 8, 2024
@plastikfan
feat(rx): add all operator (#152)
7857fc0
@plastikfan plastikfan linked a pull request Apr 8, 2024 that will close this issue
feat(rx): add all operator (#152) #153
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@plastikfan plastikfan

Labels
feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(rx): add all operator (#152) snivilised/lorax
1 participant
@plastikfan