Skip to content

Conversation

@tcoratger
Copy link
Collaborator

@tcoratger tcoratger commented Jan 24, 2026

πŸ—’οΈ Description

πŸ”— Related Issues or PRs

Followup after #321 cc @unnawut

βœ… Checklist

  • Ran tox checks to avoid unnecessary CI fails:
    uvx tox
  • Considered adding appropriate tests for the changes.
  • Considered updating the online docs in the ./docs/ directory.

@tcoratger tcoratger merged commit 87494af into leanEthereum:main Jan 24, 2026
10 checks passed
@unnawut
Copy link
Collaborator

unnawut commented Jan 24, 2026

I spotted this too while working on #321 but didn’t remove because I was hoping I could get to the point where we have test vectors for ssz and gossipsub as well, where then we need to create individual signed attestations again

but since you also have an itch on this, I agree we can remove and always add back when we actually need it

@tcoratger tcoratger added the refactor Category: refactor label Jan 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Category: refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants