Skip to content

fix: follow up to SyncChainMMR RPC#1682

Merged
drahnr merged 9 commits intonextfrom
bernhard-sync-chain-mmr-again
Feb 18, 2026
Merged

fix: follow up to SyncChainMMR RPC#1682
drahnr merged 9 commits intonextfrom
bernhard-sync-chain-mmr-again

Conversation

@drahnr
Copy link
Contributor

@drahnr drahnr commented Feb 17, 2026

Follows up to apply the following changes:

  1. add missed comments to RPC messages and docs
  2. remove stale, unused limiter
  3. remove pagination from SyncChainMMR RPC call

@drahnr drahnr added the no changelog This PR does not require an entry in the `CHANGELOG.md` file label Feb 17, 2026
@drahnr drahnr changed the title fix: add comments for SyncChainMMR RPC fix: follow up to SyncChainMMR RPC Feb 17, 2026
@drahnr drahnr requested review from bobbinth and igamigo February 17, 2026 09:12
Copy link
Collaborator

@Mirko-von-Leipzig Mirko-von-Leipzig left a comment

Choose a reason for hiding this comment

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

Just glanced through; this also fixes the merge issue from main iiuc

Copy link
Collaborator

@igamigo igamigo left a comment

Choose a reason for hiding this comment

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

LGTM!

Comment on lines 500 to 501
// Data needed to update the partial MMR from `request.block_range.block_from + 1` to
// `pagination_info.block_num`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: This still references pagination_info

Copy link
Contributor

@bobbinth bobbinth left a comment

Choose a reason for hiding this comment

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

Looks good! I left a couple of small comments inline.

@drahnr drahnr force-pushed the bernhard-sync-chain-mmr-again branch from 1afb0c8 to cd1d016 Compare February 18, 2026 16:52
@drahnr drahnr merged commit 32cf944 into next Feb 18, 2026
20 checks passed
@drahnr drahnr deleted the bernhard-sync-chain-mmr-again branch February 18, 2026 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog This PR does not require an entry in the `CHANGELOG.md` file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants