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

Add Optional type #308

Closed
dapplion opened this issue Feb 19, 2023 · 8 comments
Closed

Add Optional type #308

dapplion opened this issue Feb 19, 2023 · 8 comments

Comments

@dapplion
Copy link
Contributor

See https://eips.ethereum.org/EIPS/eip-6475

Used by Verge spec feature https://github.com/ethereum/consensus-specs/pull/3230/files#diff-96e72ffddefb3d85f86dd7cb9f7a75defae9a226db75353b2e2b5b4a01ef685dR117

@nflaig
Copy link
Member

nflaig commented Feb 19, 2023

assign me

@jflo
Copy link

jflo commented Apr 19, 2023

Are you still working on this @nflaig ? I would be willing to add to an existing branch if there is one.

@nflaig
Copy link
Member

nflaig commented Apr 19, 2023

@jflo currently not working on this, feel free to open a PR

@nflaig
Copy link
Member

nflaig commented Apr 20, 2023

Related spec update ethereum/consensus-specs#3336

@etan-status
Copy link

Note: It is still being discussed whether Optional will be more optimized, or whether it will just be a type alias for Union. See https://ethereum-magicians.org/t/eip-6475-ssz-optional/12891

@nflaig
Copy link
Member

nflaig commented Apr 20, 2023

Thanks @etan-status, will definitely follow that discussion

@nflaig
Copy link
Member

nflaig commented Sep 29, 2023

Resolved by #329

@nflaig nflaig closed this as completed Sep 29, 2023
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

No branches or pull requests

5 participants