On 03/15/2017 07:58 PM, Simon King wrote: > Hi Michael, > > On 2017-03-15, Michael Orlitzky <mich...@orlitzky.com> wrote: >> It's possible, but asking for trouble. If 1234 depends on 987, then it's >> possible that the reviewer in 987 could say "I don't like any of this, >> throw it out and start over," after which the fix in 987 might not look >> anything like what you thought it would when you fixed 1234. If 987 then >> gets set to "positive review", the release manager will merge the (now >> broken) 1234 on top of it. > > Would 1234 merge if 987 has meanwhile been totally changed? >
If 1234 changes code that 987 changes, then probably not. But if 987 adds a new function and 1234 makes use of it, then the new code will merge fine and try to use a nonexistent function. -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+unsubscr...@googlegroups.com. To post to this group, send email to sage-devel@googlegroups.com. Visit this group at https://groups.google.com/group/sage-devel. For more options, visit https://groups.google.com/d/optout.
