Bug 454372 - Proposal: switch to the cherry-pick backwards workflow
Summary: Proposal: switch to the cherry-pick backwards workflow
Status: RESOLVED FIXED
Alias: None
Product: Elisa
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR task
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-05-25 04:39 UTC by Nate Graham
Modified: 2022-06-01 20:46 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2022-05-25 04:39:14 UTC
Historically, Elisa has used a workflow whereby merge requests/commits are landed on either the master branch or the stable branch, and if the target was the stable branch because it was a bugfix, that branch is merged forward into master.

A few years ago, Plasma moved to a different workflow whereby all merge requests/commits are landed on the master branch, and cherry-picked backwards into the stable branch as needed. Having used both approaches for a few years, I find that overall the cherry-pick workflow seems to reduce friction:
1. No more asking the submitter to change the target branch of their MR if it's a bugfix
2. No more manual merging, which means no more annoying merge conflicts in the `CMakeLists.txt` file after every branching event
3. You can cherry-pick right form the GitLab web UI, which is nice and fast and you don't have to leave the website after merging using the website

I'd like to formally propose that we adopt it in Elisa.

Matthieu Gallien is Elisa's formal maintainer, but he has not been active recently so I'm also CCing the top five contributors by number of lines added over the past year according to GitHub (https://github.com/KDE/gwenview/graphs/contributors?from=2021-05-24&to=2022-05-24&type=c, which has better tools for visualizing this compared to GitLab): Me, Tranter Madi, Han Young, Friedrich Kossebau, and Laurent Montel.

What say ye?
Comment 1 trmdi 2022-05-25 04:51:16 UTC
+1
Comment 2 Friedrich W. H. Kossebau 2022-05-25 11:56:55 UTC
As I got cc::ed I have at least to say:

fix bugs in the current product version,. Only then integrate those changes also into the development version, by whatever way.

Backports to the product version instead come with increased risks of regressions and thus lower quality.

Not involved with Elisa otherwise, so unsubscribing
Comment 3 Nate Graham 2022-05-25 14:07:00 UTC
Ok, that sounds like a -1.

+1 from me, for the reasons given in the description.

Any other opinions?
Comment 4 Nate Graham 2022-06-01 20:46:32 UTC
No other responses. That's +2 and -1, which seems like a total of +1 to me, which means it's agreed-to.