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

Underlying value #7

Merged
merged 5 commits into from
Nov 19, 2021
Merged

Underlying value #7

merged 5 commits into from
Nov 19, 2021

Conversation

omeid
Copy link
Owner

@omeid omeid commented Nov 19, 2021

Allow checking if a field is set or not by exposing a IsZero.

It also drops Get() which was pretty much a useless method.

@omeid
Copy link
Owner Author

omeid commented Nov 19, 2021

Replaces #6.

@omeid omeid merged commit 09eda29 into master Nov 19, 2021
@omeid
Copy link
Owner Author

omeid commented Nov 24, 2021

Thanks @joncfoo. This has now landed.

@omeid omeid deleted the underlying-value branch January 3, 2023 05:33
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.

None yet

2 participants