Re: [PATCH] status: teach "status --short" to respect "--show-stash"

From: Junio C Hamano <gitster@pobox.com>
Date: Nov 7, 2019, 11:09 UTC
Message-ID: <xmqqimnwoscx.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <20191107103141.GA87008@helium>

"Eric N. Vander Weele" <ericvw@gmail.com> writes:

Show 12 quoted lines

>> Isn't this information available to scripts that want to read from
>> porcelain v2 output format (which is meant to be extensible by
>> allowing easy-to-parse optional headers, which this stash thing
>> exactly is).
>
> It is not available in Porcelain Format v2... yet :).
>
>
> I'm happy making '--show-stash' congruent with '--branch' in v1', which
> is would be in line with documented behavior, displaying something in
> v2 only, or both.  Let me know how you would like to proceed and I'll
> rework the patch accordingly.

Quite honestly, you do not want to ask _me_. If you left it to me, I may say that, among the possible next step you listed, the best one is to do nothing, as I do not want to see the stash info in the "status --short" output ;-)

It probably is the safest to make it available first only in v2 format. I do not know if that makes its utility too limited for the purpose of the application you have in mind.