Re: [PATCH 0/5] git check-ref-format --stdin --report-errors

2016-12-19 Thread Michael Haggerty
On 12/19/2016 07:23 PM, Junio C Hamano wrote: > Michael Haggerty writes: > >> Especially given that the output is not especially machine-readable, it >> might be more consistent with other commands to call the new feature >> `--verbose` rather than `--report-errors`. > > Don't we instead want to

Re: [PATCH 0/5] git check-ref-format --stdin --report-errors

2016-12-19 Thread Junio C Hamano
Michael Haggerty writes: > Especially given that the output is not especially machine-readable, it > might be more consistent with other commands to call the new feature > `--verbose` rather than `--report-errors`. Don't we instead want to structure the output to be machine-readable instead, giv

Re: [PATCH 0/5] git check-ref-format --stdin --report-errors

2016-12-19 Thread Ian Jackson
Michael Haggerty writes ("Re: [PATCH 0/5] git check-ref-format --stdin --report-errors"): > Thanks for your patches. I left some comments about the individual patches. Thanks for your review. > I don't know whether this feature will be popular, but it's not a lot

Re: [PATCH 0/5] git check-ref-format --stdin --report-errors

2016-12-19 Thread Michael Haggerty
On 11/04/2016 08:13 PM, Ian Jackson wrote: > I wanted to be able to syntax check lots of proposed refs quickly > (please don't ask why - it's complicated!) > > So I added a --stdin option to git-check-ref-format. Also it has > --report-errors now too so you can get some kind of useful error > mes

[PATCH 0/5] git check-ref-format --stdin --report-errors

2016-11-04 Thread Ian Jackson
I wanted to be able to syntax check lots of proposed refs quickly (please don't ask why - it's complicated!) So I added a --stdin option to git-check-ref-format. Also it has --report-errors now too so you can get some kind of useful error message if it complains. It's still not really a good bat