-
Notifications
You must be signed in to change notification settings - Fork 283
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
Make it easy to pull changes from upstream and cherry-pick celestia-core onto tendermint #528
Comments
regarding the bot, this was suggested in the past: #180 |
not only will we have to make it easy to pull changes from tendermint master, but we should also try to make it easy to port our changes to tagged versions of tendermint. This way our fork of the cosmos-sdk can replace those specific versions. ref #549 |
per celestiaorg/cosmos-sdk#34 (comment) we also need to use the same bot to keep track of the most recent release versions of tendermint and the sdk and document this somewhere clearly. |
we have made it significantly easier to upgrade from upstream, and imho can close this issue. We can still reopen this issue if other think other wise. Originally I was waiting on merging a bot that pull upstream before closing, but as shown in #558, this will still result in us having to handle conflicts manually anyway. |
* Reorganize existing guides (#492) * Moves all guides to a guids folder * Moves the actual intro to the README.md of the intro folder * Add guides to the guide README.md * Fixing links in docs/README.md (cherry picked from commit fa98b5e) # Conflicts: # docs/README.md # docs/guides/java.md # docs/guides/kotlin.md # docs/introduction/README.md # docs/introduction/what-is-cometbft.md * solving conflicts * Add guides to the guide README.md --------- Co-authored-by: Lasaro <[email protected]>
Summary
Now that celestia-core will be significantly closer to tendermint, it makes even more sense to make it easier to pull changes from tendermint. This will mainly involve reducing merge conflicts, but also include using a bot to frequently create PRs to pull in changes.
Details
It has been decided that we will do everything that we can to save time pulling changes from upstream, both in tendermint and the sdk. To further that goal, we will stop declaring the module for celestia-core, and declare it once again as tendermint. This will dramatically reduce the number of merge conflicts when pulling changes from upstream simply because the imports for both repos will be the same.
While we're doing this, we're also planning on updating to the latest version of tendermint by "resetting" (see #527) and then progressively re-implementing the remaining necessary changes back in. The idea being that this will actually be easier than merging the latest version, as we can test individual components as we add them, instead of having to do that all at once. The changes that we do add in should be a relatively straightforward process, as they are already implemented, and we have already finished work that simplifies the remaining changes mentioned in #491 (see #505). It should also further reduce the number of conflicts between the two repos.
Action Items
Core
Reset repo to upstream Reset to tendermint master #534
DataHash
to theHeader
use a bot to frequently make PRs that incorporate changes from upstream (previously discussed in git sync #180)
SDK
App
References
Huge shoutout to @marbar3778 who's previous and current efforts pulling changes from upstream were crucial in informing this decision. #526
this issue replaces #492
tracking issue #491
The text was updated successfully, but these errors were encountered: