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

Make it easy to pull changes from upstream and cherry-pick celestia-core onto tendermint #528

Closed
12 of 13 tasks
Tracked by #491
evan-forbes opened this issue Sep 15, 2021 · 4 comments
Closed
12 of 13 tasks
Tracked by #491
Assignees

Comments

@evan-forbes
Copy link
Member

evan-forbes commented Sep 15, 2021

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

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

@evan-forbes evan-forbes mentioned this issue Sep 15, 2021
31 tasks
@liamsi
Copy link
Member

liamsi commented Sep 15, 2021

regarding the bot, this was suggested in the past: #180

@evan-forbes evan-forbes self-assigned this Sep 17, 2021
@evan-forbes evan-forbes changed the title Make it easy to pull changes from upstream Make it easy to pull changes from upstream and rebase cherry-pick celestia-core onto tendermint Sep 17, 2021
@evan-forbes evan-forbes changed the title Make it easy to pull changes from upstream and rebase cherry-pick celestia-core onto tendermint Make it easy to pull changes from upstream and cherry-pick celestia-core onto tendermint Sep 17, 2021
@evan-forbes
Copy link
Member Author

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

@evan-forbes
Copy link
Member Author

evan-forbes commented Sep 30, 2021

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.

@evan-forbes
Copy link
Member Author

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.

evan-forbes pushed a commit that referenced this issue Jun 9, 2023
* 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 <lasaro@informal.systems>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants