Re: [PATCH 3/4] submodule--helper: fix BUG message in ensure_core_worktree
- From
Stefan Beller <sbeller@google.com>
- Date
- Dec 12, 2018, 22:46 UTC
- Message-ID
- <CAGZ79kb0Vqk8Gtao6OdKx7gJi6pCEpLzcqQsk=uqCLfePZrmVw@mail.gmail.com>
- In-Reply-To
- <xmqqsgz87bj9.fsf@gitster-ct.c.googlers.com>
Show 6 quoted lines
> Unlike the step 2/4 I commented on, this does explain what this > wants to do and why, at least when looked from sideways. Is the > above saying the same as the following two-liner? > > An ealier mistake while rebasing to produce 74d4731da1 > failed to update this BUG message. Fix this.
I am not sure if it was rebasing, which was executed mistakenly. So maybe just saying "74d4731da1 contains a faulty BUG message. Fix it." would do.
The intent of the longer message was to shed light in how I found the BUG (ie. I did not see the BUG message, which would ask me to actually fix a bug, but found it via code inspection), which I thought was valuable information, too.