Replies: 2 comments
-
Added a diffview page so that we can keep an eye on the fork diff size: https://scroll-tech.github.io/reth
It will be a bit tricky to then submit this abstraction back to upstream, since the |
Beta Was this translation helpful? Give feedback.
0 replies
-
Here is the current process to sync with upstream:
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Overview
Our ambitions with this project are to implement a scroll node using reth in an SDK like manner. As reth may not provide the abstractions we require to achieve an SDK like implementation from day one we may find that we have to introduce features on this fork of reth. However, we want these features to be short lived and our ambition is to push the required abstractions upstream such that we can remove the diffs in a timely fashion.
Branching Strategy
The branching strategy we will adopt is intended to minimise the diff and maintain in sync with upstream as much as possible. The branching strategy will only cover the development phase of the project, as we approach our first major version release we should update the strategy to support releases.
In our branching strategy we will introduce a new branch
scroll
. This branch will act as the main branch from which features will be developed. As a feature development begins a new feature branch will be created from thescroll
branch. Themain
branch will be kept in sync withmain
from upstream and we will periodically mergemain
intoscroll
. We must define the triggers and frequency in which we merge upstream intoscroll
. A simple proposal would be to sync at least every 2 weeks. If we have to merge before the 2 week threshold then the timer will reset at the time of the merge.Development Process
In the circumstance that we need to introduce a new feature to reth which requires us to diverge from upstream we should develop and abstraction that allows us to remove the diff before we move onto the next feature.
Beta Was this translation helpful? Give feedback.
All reactions