<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 25, 2020 at 3:46 PM Bhushan Shah <<a href="mailto:bshah@mykolab.com">bshah@mykolab.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello everyone!<br>
<br>
This is a proposal to change our workflow regarding release branches and<br>
our merge workflow,<br>
<br>
# Current workflow<br>
<br>
- Current workflow is that we commit to stable branch and then merge it<br>
  upwords until master branch<br>
- i.e commit to Plasma/5.18 branch, merge 5.18 into 5.19 and then<br>
  master<br>
<br>
# Proposed workflow<br>
<br>
- Proposed workflow is to instead commit all changes in master, and<br>
  cherry-pick related changes in the stable branch as needed<br>
<br></blockquote><div><br></div><div>Gitlab supposedly has a magic button for it just after a commit has landed in master.</div><a href="https://docs.gitlab.com/ee/user/project/merge_requests/cherry_pick_changes.html">https://docs.gitlab.com/ee/user/project/merge_requests/cherry_pick_changes.html</a><div><br></div><div>Though I don't know how to trigger that in our UI.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
# Why?<br>
<br>
We occasionally hit several issues with current workflow,<br>
<br>
- Merge conflicts when merging changes upwords<br>
- Changes which are valid only for stable branch needs to be reverted in<br>
  master branches<br>
- Accidential merges from the master branch to stable branch which needs<br>
  to be force-pushed<br>
<br></blockquote><div>It's worth noting that Qt also recently changed to merge to dev, cherry-pick backwards. <br></div><div></div><div>In principle I completely agree. <br></div><div><br></div><div>I have a few fears, which I hope are all addressable.</div><div><br></div><div>  - we need some reference to the real commit. <br></div><div><br></div><div>Qt adds</div><div>"    (cherry picked from commit 6de0287d7c3aa4251fe6eb4f970d73ce11cf07fc)" to the commit message automagically somehow in their workflow.<br></div><div><br></div><div>  - without making people commit locally into stable, could it encourage people to not test as much?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
For now I am proposing this change only for Plasma repositories if we<br>
like it, we can propose this workflow for rest of KDE repositories, but<br>
that needs discussions in kde-devel/kde-core-devel separately.<br>
<br>
Thoughts?<br></blockquote><div><br></div><div>I like consistency across KDE, otherwise it's very difficult for people who contribute in N places.</div><div>We should at least email before we change, though we can still discuss things here first.<br></div><div></div><div><br></div></div><div class="gmail_quote">David<br></div><br></div>