[gmx-developers] Unsquashed MR in master
mark.j.abraham at gmail.com
Wed Oct 6 07:08:36 CEST 2021
I think it's clear to follow normal practice and not rewrite history. Doing
so means we have to rename master, delete it, push a fixed one, organize to
replace the updated content, and then fix/delete the repo of anybody who's
pulled in the meantime, including projects that automatically mirror us
(including our own mirror on github). That's a small amount of activity for
some people. Doing nothing means that there's a chance in future of someone
doing a bisection and hitting one of these 24 commits that might not build
or might fail a test. If they do, most of the time they'll see that the
failure mode is different and they can move to a different nearby commit
and try again. This is already what they would have to do if the case
they're investigated was something that only failed in post-submit testing
or the pre-submit testing that is not a formal requirement for gitlab to
permit an MR to be submitted.
On Tue, 5 Oct 2021 at 18:57, Magnus Lundborg <magnus.lundborg at scilifelab.se>
> Hi everyone,
> By mistake I merged an MR without squashing all commits today. The
> question now is if we should try to restore the master branch and
> rewrite the history or if we should let it be. Leaving it as it is might
> cause trouble with future 'git bisect' since some commits in between
> might not compile or might fail tests. But on the other hand, rewriting
> history is not optimal either, especially since there is at least one
> commit merged afterwards. I think it could be good not to merge anything
> more until we have decided how to proceed. What do you think would be best?
> I'm sorry about this mess.
> Gromacs Developers mailing list
> * Please search the archive at
> http://www.gromacs.org/Support/Mailing_Lists/GMX-developers_List before
> * Can't post? Read http://www.gromacs.org/Support/Mailing_Lists
> * For (un)subscribe requests visit
> or send a mail to gmx-developers-request at gromacs.org.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the gromacs.org_gmx-developers