Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feature: graft auto sync #5633

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feature: graft auto sync #5633

wants to merge 1 commit into from

Conversation

dwerner
Copy link
Contributor

@dwerner dwerner commented Sep 5, 2024 •
edited
Loading

This PR is intended to:

  1. Implement auto grafting via a recursive crawling of manifests and syncing.
  2. Provide a runner test that can exercise and test that functionality.

As a side-effect, I needed to catch calls to some internals of our test chain, and to do that required newtyping Chain in WrappedEthChain.

@dwerner dwerner force-pushed the feature/auto-resolve-sub-grafts branch 8 times, most recently from 3acbeb3 to 478132e Compare September 6, 2024 21:43
@fordN fordN added the enhancement New feature or request label Sep 9, 2024
@dwerner dwerner force-pushed the feature/auto-resolve-sub-grafts branch 2 times, most recently from f7bd5ca to a83082a Compare September 11, 2024 23:30
@dwerner dwerner force-pushed the feature/auto-resolve-sub-grafts branch 6 times, most recently from 0a452d2 to 325931a Compare November 13, 2024 19:46
@dwerner dwerner force-pushed the feature/auto-resolve-sub-grafts branch from 325931a to 119438e Compare November 13, 2024 20:00
@dwerner dwerner marked this pull request as ready for review November 13, 2024 20:09
@dwerner dwerner requested review from fordN and mangas November 13, 2024 20:12
@mangas mangas removed their request for review February 3, 2025 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fordN fordN Awaiting requested review from fordN

At least 1 approving review is required to merge this pull request.

Assignees

@dwerner dwerner

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dwerner @fordN