Justin Traglia
Justin Traglia
@koraykoska @michaelsproul we ran into the same issue with Reth. This was our fix: * https://github.com/paradigmxyz/reth/pull/7123
Ah I had a partial build so I didn't notice the other instances of `ordinal()`. Marking as a draft.
> @zilm13: I was also thinking on checking the order, but decided it's too paranoid for now. I also had this thought. I checked the specs and didn't see anything...
I think that looks good! But maybe we do need to check order after all. Chatting with colleagues some about it and it seems that it does matter. Checking order...
Made a PR to the consensus specs to bring this up: * https://github.com/ethereum/consensus-specs/pull/3544
I've pulled in changes from master & addressed Stefan's review comments. If the team is still interested in merging this PR, it should be ready now. If not, happy to...
Hey @ralexstokes, yeah that's my fault. I forgot to update the version from v1.0.0 to v1.0.1 in Cargo.toml, so we were unable to publish the new version. I've been waiting...
@ralexstokes we just made the v1.0.2 release. I've pinged @Rjected about publishing the Rust package.
@ralexstokes here's the link: https://crates.io/crates/c-kzg/1.0.2
I think this is a good idea. @asn-d6 @pawanjay176 what do y'all think about this?