celestiaorg / celestia-core

A fork of CometBFT
Apache License 2.0
481 stars 264 forks source link

ci: add ability to backport to main #1398

Closed rootulp closed 3 months ago

rootulp commented 3 months ago

Motivation

I just merged two PRs to v0.34.x-celestia that should probably be backported to main if we're still maintaining main.

cmwaters commented 3 months ago

main is still the default branch so backporting would mean to go from main to v0.34.x-celestia. Eventually we will need a branch to target breaking changes (and a "v2" consensus) but It's unclear to me at the moment as to whether we want to maintain main in the interim

rootulp commented 3 months ago

but It's unclear to me at the moment as to whether we want to maintain main in the interim

+100

We only need this PR if we plan on maintaining main and v0.34.x-celestia.

Tangent: I'm misusing the term "backport" in this PR. It may be clearer to call it "forwardport" from v0.34.x-celestia to main.

cmwaters commented 3 months ago

I don't mind merging this, I just think for now the proper flow should still be merging to main and backporting to v0.34.x