Skip to content

Remove checkpoint sync from incoming features#138

Merged
ilitteri merged 1 commit intomainfrom
update-readme-incoming-features
Feb 19, 2026
Merged

Remove checkpoint sync from incoming features#138
ilitteri merged 1 commit intomainfrom
update-readme-incoming-features

Conversation

@pablodeymo
Copy link
Collaborator

Motivation

Checkpoint sync for long-lived networks (#80) has been implemented, so it should no longer appear in the incoming features list.

Description

Remove the checkpoint sync entry from the "Incoming features" section in the README.

Checkpoint sync for long-lived networks has been implemented (#80),
so remove it from the incoming features list in the README.
@github-actions
Copy link

🤖 Kimi Code Review

Review Summary

The change is minimal and only affects documentation. The removal of "Checkpoint sync for long-lived networks" from the TODO list in README.md appears to be a simple documentation update.

No code issues found - this is purely a documentation change that removes a completed feature from the roadmap.


Automated review by Kimi (Moonshot AI) · custom prompt

@github-actions
Copy link

🤖 Codex Code Review

No code changes in this PR; it only removes a single bullet from the roadmap in README.md:102. There are no impacts on correctness, security, performance, consensus logic, or SSZ/XMSS paths.

Findings:

  • None.

Notes:

  • Since this is documentation-only, no tests or additional review of consensus logic applies.

If you want me to check anything beyond the diff (e.g., for unstaged changes), point me to the files.


Automated review by OpenAI Codex · custom prompt

@github-actions
Copy link

🤖 Claude Code Review

This is a trivial one-line documentation change — removing a completed feature from the "Incoming features" list in README.md.


PR #138 — Remove checkpoint sync from incoming features

Change: Removes one line from README.md:102 — the "Checkpoint sync for long-lived networks" entry from the incoming features list, since issue #80 has been implemented.

Assessment: The change is correct and appropriate. Removing completed work from a roadmap list is good housekeeping. The diff is exactly as described in the PR, with no functional code changes.

No issues found. This is ready to merge.


Automated review by Claude (Anthropic) · custom prompt

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 19, 2026

Greptile Summary

Removed checkpoint sync from the "Incoming features" section since it has been implemented. The feature was added in PR #93 and documented in PR #120, so it's correct to remove it from the planned features list.

  • Checkpoint sync implementation exists in bin/ethlambda/src/checkpoint_sync.rs
  • Documentation exists in docs/checkpoint_sync.md
  • Removal from incoming features is appropriate and accurate

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • This is a simple documentation update that accurately removes an implemented feature from the planned features list. The checkpoint sync feature is confirmed to be implemented in the codebase with full implementation and documentation.
  • No files require special attention

Important Files Changed

Filename Overview
README.md Removed checkpoint sync from incoming features list as it has been implemented

Last reviewed commit: 36d6329

@ilitteri ilitteri merged commit 0f5ea11 into main Feb 19, 2026
7 checks passed
@ilitteri ilitteri deleted the update-readme-incoming-features branch February 19, 2026 14:56
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.

2 participants

Comments