Skip to content

Conversation

@macladson
Copy link
Member

Proposed Changes

Remove more of the temporary re-exports from consensus/types

Additional Info

Previous PR: #8643

@macladson macladson added ready-for-review The code is ready for review code-quality low-hanging-fruit Easy to resolve, get it before someone else does! labels Jan 15, 2026
Copy link
Member

@pawanjay176 pawanjay176 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this doesn't change any logic, LGTM.

@pawanjay176 pawanjay176 added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Jan 16, 2026
@mergify mergify bot added the queued label Jan 16, 2026
@mergify
Copy link

mergify bot commented Jan 16, 2026

Merge Queue Status

✅ The pull request has been merged at f3238c7

This pull request spent 1 hour 1 minute 52 seconds in the queue, including 1 hour 35 seconds running CI.
The checks were run on draft #8670.

Required conditions to merge
  • check-success=local-testnet-success
  • check-success=test-suite-success

mergify bot added a commit that referenced this pull request Jan 16, 2026
mergify bot added a commit that referenced this pull request Jan 16, 2026
@mergify mergify bot merged commit 3903e1c into sigp:unstable Jan 16, 2026
36 checks passed
@mergify mergify bot removed the queued label Jan 16, 2026
@macladson macladson deleted the more-consensus-types-reexport-cleanup branch January 16, 2026 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-quality low-hanging-fruit Easy to resolve, get it before someone else does! ready-for-merge This PR is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants