Re: [PATCH 3/6] push: change `simple` to accommodate triangular workflows
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jun 24, 2013, 07:59 UTC
- Message-ID
- <7vppvcc51j.fsf@alter.siamese.dyndns.org>
- In-Reply-To
- <CALKQrgfAT9GhD-_tZHr9wRA6R6g7ttDWTsmUQXdWUhG-gwNBqQ@mail.gmail.com>
Johan Herland <johan@herland.net> writes:
Show 6 quoted lines
>> An earlier round of this change by mistake broke the safety for >> "simple" mode we have had since day 1 of that mode to make sure that >> the branch in the repository we update is set to be the one we fetch >> and integrate with, but it has been fixed. > > Shouldn't there be an acompanying test to demonstrate this mistake being fixed?
An operation that has to expect failure due to safety was disabled by the broken version. The squashed end result reverts that change to the test, to make sure we did not break the safety.