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

Hide PollState variants #92

Merged

yoshuawuyts merged 1 commit into yoshuawuyts:main from matheus-consoli:pollstate-set-ready-set-consumed

Nov 17, 2022

Collaborator

matheus-consoli commented Nov 16, 2022

Closes #88

I also "hid" PollState::Ready behind a set_ready. Now no one outside utils/poll_state/poll_state.rs references the variants by name.


          Hide PollState variants

d048599

yoshuawuyts approved these changes

View reviewed changes

Owner

yoshuawuyts left a comment

Thanks, this is great!

yoshuawuyts merged commit 330c8ee into yoshuawuyts:main

matheus-consoli deleted the pollstate-set-ready-set-consumed branch

November 17, 2022 01:39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet