Skip to content

test: fix ssz types in fork_choice spec tests - #6767

Merged
ensi321 merged 1 commit into
electra-forkfrom
nflaig/fork-choice-spec
May 14, 2024
Merged

test: fix ssz types in fork_choice spec tests#6767
ensi321 merged 1 commit into
electra-forkfrom
nflaig/fork-choice-spec

Conversation

@nflaig

@nflaig nflaig commented May 14, 2024

Copy link
Copy Markdown
Member

Motivation

Fixes issue brought up by @jeluard in fork choice spec tests

First offset must equal to fixedEnd 229 != 228

Description

Select correct ssz types for Attestation and AttesterSlashing in fork_choice spec tests

@nflaig
nflaig requested a review from a team as a code owner May 14, 2024 07:34
@ensi321
ensi321 merged commit 140a25b into electra-fork May 14, 2024
@ensi321
ensi321 deleted the nflaig/fork-choice-spec branch May 14, 2024 07:37
g11tech pushed a commit to g11tech/lodestar that referenced this pull request Jun 19, 2024
@wemeetagain

Copy link
Copy Markdown
Member

馃帀 This PR is included in v1.22.0 馃帀

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.

4 participants