Skip to content

Conversation

JonathanOppenheimer
Copy link
Member

Why this should be merged

This is the second split up this PR #1679 as requested by @ceyonur. This works towards the goal of aligning the two repositories to make the transition to the coreth sync refactor simpler, and to the eventual merger of all of these files.

@JonathanOppenheimer JonathanOppenheimer self-assigned this Sep 22, 2025
@JonathanOppenheimer JonathanOppenheimer requested a review from a team as a code owner September 22, 2025 19:08
@JonathanOppenheimer JonathanOppenheimer changed the base branch from JonathanOppenheimer/just-wrapped-block to master September 22, 2025 19:09
@JonathanOppenheimer JonathanOppenheimer added the DO NOT MERGE This PR is not meant to be merged in its current state label Sep 22, 2025
@JonathanOppenheimer
Copy link
Member Author

This PR is blocked by #1742. #1742 MUST be merged into master and reflected here before this can be reviwed.

@JonathanOppenheimer JonathanOppenheimer removed the DO NOT MERGE This PR is not meant to be merged in its current state label Sep 25, 2025
@JonathanOppenheimer
Copy link
Member Author

This PR is now R4R

Co-authored-by: Austin Larson <78000745+alarso16@users.noreply.github.com>
Signed-off-by: Jonathan Oppenheimer <147infiniti@gmail.com>
Copy link
Contributor

@alarso16 alarso16 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 to me, but I think @ceyonur should approve too

@JonathanOppenheimer JonathanOppenheimer added the state-sync-migration Migration of state sync code to `avalanchego`. label Oct 6, 2025
Signed-off-by: Jonathan Oppenheimer <jonathan.oppenheimer@avalabs.org>
JonathanOppenheimer and others added 2 commits October 8, 2025 13:51
Co-authored-by: Ceyhun Onur <ceyhun.onur@avalabs.org>
Signed-off-by: Jonathan Oppenheimer <147infiniti@gmail.com>
ceyonur
ceyonur previously requested changes Oct 9, 2025
Copy link
Collaborator

@ceyonur ceyonur left a comment

Choose a reason for hiding this comment

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

just the clock stuff

@JonathanOppenheimer
Copy link
Member Author

just the clock stuff

Gotcha, thanks for the feedback, should be g2g now.

@ceyonur ceyonur added this pull request to the merge queue Oct 10, 2025
Merged via the queue into master with commit 27d9d53 Oct 10, 2025
13 checks passed
@ceyonur ceyonur deleted the JonathanOppenheimer/just-sync-changes branch October 10, 2025 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

coreth sync state-sync-migration Migration of state sync code to `avalanchego`.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants