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

Mention any_of as alternative to enum #2119

Merged
merged 2 commits into from
Dec 4, 2024

Conversation

yannham
Copy link
Member

@yannham yannham commented Dec 3, 2024

Prompted by #1993 (comment).

Since enum variants can't be (de)serialized, they aren't good candidates for values that can go in a configuration. This commit makes the documentation on this point clearer, and points to the alternative untagged union std.contract.any_of.

I can't request reviews for people outside of the Tweag org, but cc @rben01 if you'd like to have a look.

Since enum variants can't be (de)serialized, they aren't good candidates
for values that can go in a configuration. This commit makes the
documentation on this point clearer, and points to the alternative
untagged union `std.contract.any_of`.
@yannham yannham requested a review from jneem December 3, 2024 10:23
Copy link
Contributor

github-actions bot commented Dec 3, 2024

@yannham yannham added this pull request to the merge queue Dec 4, 2024
Merged via the queue into master with commit bbaa956 Dec 4, 2024
5 checks passed
@yannham yannham deleted the doc/enum-variants-mention-anyof branch December 4, 2024 09:27
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.

2 participants