[Paraview-developers] [EXTERNAL] Re: Dashboard question

Marcus D. Hanwell marcus.hanwell at kitware.com
Fri Mar 2 10:07:47 EST 2012


+1 from me. Once you have published a branch you should avoid
rebasing. Add new commits to the topic, push and merge and move on.
Once it is clean the gate keeper review can merge the whole thing to
master.

My $0.01...

On Fri, Mar 2, 2012 at 10:05 AM, Moreland, Kenneth <kmorel at sandia.gov> wrote:
> Rebasing is evil.  Capture your small fixes on topic branches and wear
> your mistakes with pride.
>
> That's my $0.02.
>
> -Ken
>
>
> On 3/2/12 8:01 AM, "Biddiscombe, John A." <biddisco at cscs.ch> wrote:
>
>>Kyle,
>>
>>Thanks
>>
>>>
>>There is no way to un-merge but you can revert your commit and merge that
>>into next.
>><
>>
>>Does that mean that next is continually growing with commits and reverted
>>commits and if you merged a huge load of stuff by mistake, you can't
>>reset next back to the point before you added a lot of commits to clean
>>it up.
>>
>>My hypothetical use case was that you merge to next, but get some
>>unexpected problems, so you end up doing a whole bunch of small commits
>>to fix the issues, afterwards you want to rebase all those branch commits
>>into a single clean commit and merge that to next instead of all the duff
>>ones.
>>
>>Am I to understand that that's not possible in quite the way I imagined.
>>If not, then I'll be extra careful about stage merge.
>>
>>JB
>>
>>
>>_______________________________________________
>>Paraview-developers mailing list
>>Paraview-developers at paraview.org
>>http://public.kitware.com/mailman/listinfo/paraview-developers
>>
>
>
> _______________________________________________
> Paraview-developers mailing list
> Paraview-developers at paraview.org
> http://public.kitware.com/mailman/listinfo/paraview-developers


More information about the Paraview-developers mailing list