The issue with rebasing is that if you release code publicly, other people can clone it, then the rebase changes the history but the history isn't changed on everyone else's clone so things get harry. Rebasing changes the history, so it should only be done when you are the only person who's "seen" the history. It also screws up all the comments in the Github PR because they can no longer link to an existing diff. But if you are the *only* person working on a branch/pr that you've push publicly then rebasing a bunch of small commits into fewer nicely packaged commits is fine to do before merging to master. Just beware that once you start working on a branch with commits by more than one author headaches will likely be introduced by rebasing.
Jason moorepants.info +01 530-601-9791 On Wed, Oct 1, 2014 at 4:46 PM, Colin Macdonald <macdon...@maths.ox.ac.uk> wrote: > [replying to maillist instead of issue tracker] > > > <https://github.com/cbm755>I've noticed that you rebase a lot. Rebasing > after code is "public" is not a great > > habit to get into. Check our dev workflow ... > > Thanks for this. From the dev workflow doc: > > > The only time when it is recommended to rebase instead of merge is when > you > > need to edit your commit messages, or **remove unnecessary commits**. > > Would this latter case include cleaning up a (sordid) history that has > approaches > tried but latter reverted? For example in pr #7824, I had some low-level > changes > to superclasses that are later reverted. Otherwise, strikes me as messy > to have > that sort of thing in the history of master... > > thanks, > Colin > > -- > You received this message because you are subscribed to the Google Groups > "sympy" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to sympy+unsubscr...@googlegroups.com. > To post to this group, send email to sympy@googlegroups.com. > Visit this group at http://groups.google.com/group/sympy. > To view this discussion on the web visit > https://groups.google.com/d/msgid/sympy/c94ed790-6016-4e26-a562-e778cb70d412%40googlegroups.com > <https://groups.google.com/d/msgid/sympy/c94ed790-6016-4e26-a562-e778cb70d412%40googlegroups.com?utm_medium=email&utm_source=footer> > . > For more options, visit https://groups.google.com/d/optout. > -- You received this message because you are subscribed to the Google Groups "sympy" group. To unsubscribe from this group and stop receiving emails from it, send an email to sympy+unsubscr...@googlegroups.com. To post to this group, send email to sympy@googlegroups.com. Visit this group at http://groups.google.com/group/sympy. To view this discussion on the web visit https://groups.google.com/d/msgid/sympy/CAP7f1AiSBVohxRJaG%3DKtpKfao43KH_uX7HS0%2BQoRozfKC9qz5A%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.
