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.

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

Rename chainsync to legacychainsync #541

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

Conversation

ezdac
Copy link
Collaborator

@ezdac ezdac commented Oct 18, 2024

1/4 - Rename chainsync to legacychainsync

This is part of a series of 4 Pull Requests that all aim to refactor our old
chainsync mechanism. The reason to split this up is to provide more digestible
and grouped parts of the refactoring effort. I am aware this is a big change,
but especially for refactoring an integral part of the system, and designing
it at the same time, at least for me it is impossible to provide smaller
increments of work that don't leave the code in an uncompilable,
non-functional state.

The chainsync mechanism will be refactored in the following PRs.
It will temporarily be still in use in the follow-up PRs of the refactoring, and the
optimism keyper package will still use the legacychainsync. This is because integrating the
new chainsync mechanism in the optimism keyper was currently no priority.

@ezdac ezdac force-pushed the ezdac/01-remove-optimism branch from 0f09d21 to ba7695d Compare October 25, 2024 20:04
@ezdac ezdac changed the title chore: remove optimism package Rename chainsync to legacychainsync Oct 25, 2024
@ezdac ezdac marked this pull request as ready for review October 25, 2024 23:50
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.

1 participant