Subject: Re: What's cooking in git.git (Nov 2019, #03; Tue, 19)

From: Denton Liu <[email-masked]>
Date: Wed, 20 Nov 2019 15:32:14 -0800

Hi Junio,

On Tue, Nov 19, 2019 at 04:05:48PM +0900, Junio C Hamano wrote:

[...]

> * dl/pretty-reference (2019-11-19) 11 commits
>  - squash! pretty: implement 'reference' format
>  - SubmittingPatches: use `--pretty=reference`
>  - pretty: implement 'reference' format
>  - pretty: provide short date format
>  - t4205: cover `git log --reflog -z` blindspot
>  - pretty.c: inline initalize format_context
>  - revision: make get_revision_mark() return const pointer
>  - completion: complete `tformat:` pretty format
>  - SubmittingPatches: remove dq from commit reference
>  - pretty-formats.txt: use generic terms for hash
>  - SubmittingPatches: use generic terms for hash
> 
>  "git log" family learned "--pretty=ref" that gives the name of a

Since this will be used for release notes, could we use the full name of
the pretty format (i.e. "--pretty=reference")?

(Also, TIL that --pretty can match incomplete prefixes!)

>  commit in the format that is often used to refer to it in log
>  messages.
> 
>  Expecting finishing touches to the data handling.
>  Other than that, this is reasonably well done, I think.

[...]

> * dl/t5520-cleanup (2019-11-19) 14 commits
>  - t5520: replace `! git` with `test_must_fail git`
>  - t5520: remove redundant lines in test cases
>  - t5520: replace $(cat ...) comparison with test_cmp
>  - t5520: don't put git in upstream of pipe
>  - t5520: test single-line files by git with test_cmp
>  - t5520: use test_cmp_rev where possible
>  - t5520: replace test -{n,z} with test-lib functions
>  - t5520: use test_line_count where possible
>  - t5520: remove spaces after redirect operator
>  - t5520: replace test -f with test-lib functions
>  - t5520: let sed open its own input
>  - t5520: use sq for test case names
>  - t5520: improve test style
>  - t: teach test_cmp_rev to accept ! for not-equals
> 
>  Test cleanup.
> 
>  Will merge to 'next'.

Any chance that we can queue this patch[1] either as a separate series
or as a precursor to the above?

Thanks,

Denton

[1]: https://public-inbox.org/git/20191114005215.GA24527@generichostname