List for cgit developers and users
 help / color / mirror / Atom feed
From: Jason at zx2c4.com (Jason A. Donenfeld)
Subject: [PATCH] tests: use Git's test framework
Date: Sun, 7 Apr 2013 17:28:59 +0200	[thread overview]
Message-ID: <CAHmME9oYJRc0uJ_EED1CY2_Rz0t4tiY+ooqdiuzw1-mbw6_bxA@mail.gmail.com> (raw)
In-Reply-To: <43a3cf29f2cfe519b86396aaf86288e87f237776.1364825141.git.john@keeping.me.uk>

On Mon, Apr 1, 2013 at 4:09 PM, John Keeping <john at keeping.me.uk> wrote:
> Obviously there are a number of further changes that can be done on top
> of this to avoid the now-unnecessary wrapper functions and use a test
> prerequisite for HTML tidy but I don't want to work on those unless
> there is agreement that this is a sensible direction.

I'm okay with merging this patch. But before I merge from wip to
master, it'd be nice to see the further changes -- eliminating a few
wrapper functions, using test prerequisites, etc. I applied this on
top of yours --
http://git.zx2c4.com/cgit/commit/?h=wip&id=2b9829453790425c98d3ad7427f82ba9f6446d01
-- but this is just a start.




  parent reply	other threads:[~2013-04-07 15:28 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-01 14:09 john
2013-04-04 14:24 ` Jason
2013-04-04 14:34   ` john
2013-04-04 14:32 ` Jason
2013-04-04 16:19   ` mailings
2013-04-07 15:28 ` Jason [this message]
2013-04-08 14:28   ` Jason

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAHmME9oYJRc0uJ_EED1CY2_Rz0t4tiY+ooqdiuzw1-mbw6_bxA@mail.gmail.com \
    --to=cgit@lists.zx2c4.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).