Hi all, Last year, we had a discussion on rebase and squash merge PRs to the develop branch,
* to have linear git history * help with bisecting * easier root cause analysis of failures * easier to backport changes However, many developers have reached out mentioning that sometimes they have clicked the merge button in hurry when they meant to rebase/squash. Once clicked there is no going back. All of them suggested that to have the GitHub setting on develop branch to rebase and squash merge option and hence there is no room for mistakes to occur. I would like to propose to that we set the GitHub setting on develop PR to rebase and squash buttons. Please do note that this is reversible and can be changed back, hence there is no harm in trying it out. Regards Nabarun Nag ________________________________ From: Owen Nichols (Pivotal) <onich...@pivotal.io> Sent: Thursday, April 9, 2020 11:45 AM To: dev@geode.apache.org <dev@geode.apache.org> Subject: Re: [REQUEST] Squash merge please I’ve noticed quite a few PRs in the last week that were merged with “Merge” rather than “Squash and Merge”. While the community consensus was to continue to allow all merge options, please try to default to “Squash and Merge” whenever you can to keep history as linear as possible. GitHub will save the last method you used in a cookie, which helps, but then it’s easy to miss when it resets itself back to the default of “Merge” some months later because you cleared cookies, changed browsers, etc. > On Oct 22, 2019, at 5:12 PM, Nabarun Nag <n...@pivotal.io> wrote: > > Hi Geode Committers, > > A kind request for using squash commit instead of using merge. > This will really help us in our bisect operations when a regression/flakiness > in the product is introduced. We can automate and go through fewer commits > faster, avoiding commits like "spotless fix" and "re-trigger precheck-in" or > other minor commits in the merged branch. > > Also, please use the commit format : (helps us to know who worked on it, what > is the history) > GEODE-xxxx: <brief intro > > > * explanation line 1 > * explanation line 2 > > This is not a rule or anything, but a request to help out your fellow > committers in quickly detecting a problem. > > For inspiration, we can look into Apache Kafka / Spark where they have a > complete linear graph for their main branch HEAD [see attachment] > > Regards > Naba. > >
