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

Is Builder intended to be included in the library? #20

Closed
kaishin opened this issue Jan 3, 2020 · 2 comments
Closed

Is Builder intended to be included in the library? #20

kaishin opened this issue Jan 3, 2020 · 2 comments

Comments

@kaishin
Copy link

kaishin commented Jan 3, 2020

Following the examples in my own project, I realized that the Builder helper protocol is not part of the library. Is that an oversight or should I just copy the set method directly in my own project?
@sergdort
Copy link
Owner

Hey @kaishin I've now changed Reducer signature from (State, Event) -> State to be (inout State, Event) -> Void so there is no need for Builder helper anymore, just make properties of your State struct var is they intent to be mutable

@kaishin
Copy link
Author

kaishin commented Feb 20, 2020

Thanks!

Feel free to close or leave the issue open as you see fit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kaishin @sergdort