Re: [PATCH 3/3] doc: merge-tree: use || directly

From: Kristoffer Haugsbakk <kristofferhaugsbakk@fastmail.com>
Date: Oct 7, 2024, 15:44 UTC
Message-ID: <5cfe7170-3fd5-432f-b6af-702b8cb2f611@app.fastmail.com>
In-Reply-To: <CABPp-BFG0KQVzu_AT-GwW0rg0Ly3uVY1+M4Sgh5E8Gn9A=5BKQ@mail.gmail.com>

On Mon, Oct 7, 2024, at 17:24, Elijah Newren wrote:

Show 15 quoted lines

> On Mon, Oct 7, 2024 at 4:11 AM Kristoffer Haugsbakk
> <code@khaugsbakk.name> wrote:
>>
>> From: Kristoffer Haugsbakk <kristofferhaugsbakk@fastmail.com>
>>
>> From: Kristoffer Haugsbakk <code@khaugsbakk.name>
>>
>> Use `||` directly since that is more straightforward than checking the
>> last exit status.
>>
>> Also use `echo` and `exit` since `die` is not defined.
>
> Not defined where?  It's defined in my scripts...  ;-)
>
> Just kidding; this is a good change.

:D

Thanks for taking a look! I’ll send the second version tomorrow.

-- 
Kristoffer Haugsbakk