git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] push --force-with-lease: Fix ref status reporting

From
Andrew Wheeler <awheeler@motorola.com>
Date
Jan 26, 2016, 02:36 UTC
Message-ID
<CAE=k0mt=wCA+evSXdDp1w5n5r34pyNoKvz14wfLdwzVTS+jGVQ@mail.gmail.com>
In-Reply-To
<xmqqvb6hfo1u.fsf@gitster.mtv.corp.google.com>
On Mon, Jan 25, 2016 at 1:37 PM, Junio C Hamano <gitster@pobox.com> wrote:
> >                *     passing the --force argument
> > +              *
>
> This new blank line is probably unwanted.
Show 5 quoted lines
> Do we want to make sure that other people will not break this fix in
> the future by adding a few tests, perhaps to t/t5533?
>
> Thanks.
>

Great idea. I'll add some tests to t/t5533, remove that blank line above, and share a new patch.

Thanks for the review!
-andrew
Previous: Junio C Hamano
Message 3 of 3 in “push --force-with-lease: Fix ref status reporting”
  1. push --force-with-lease: Fix ref status reportingAndrew Wheeler, Jan 21, 2016
  2. Junio C HamanoJan 25, 2016
  3. Andrew WheelerJan 26, 2016

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.