search

ChainSafe / ssz

Typescript implementation of Simple Serialize (SSZ)
https://simpleserialize.com/
Other
50 stars 19 forks source link

Add Optional type #308

Closed dapplion closed 1 year ago

dapplion commented 1 year ago

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 commented 1 year ago

assign me

jflo commented 1 year ago

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

nflaig commented 1 year ago

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

nflaig commented 1 year ago

Related spec update https://github.com/ethereum/consensus-specs/pull/3336

etan-status commented 1 year ago

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 commented 1 year ago

Thanks @etan-status, will definitely follow that discussion

nflaig commented 1 year ago

Resolved by https://github.com/ChainSafe/ssz/pull/329

draghtnight commented 8 months ago

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

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