Re: [GSoC][PATCH] tests: avoid using pipes
- Date: Sun, 10 Mar 2019 16:05:43 +0100
- From: Christian Couder <christian.couder@xxxxxxxxx>
- Subject: Re: [GSoC][PATCH] tests: avoid using pipes
On Sun, Mar 10, 2019 at 9:28 AM ttjtftx <ttjtftx@xxxxxxxxx> wrote:
> On Sun, Mar 10, 2019 at 2:06 PM Christian Couder
> <christian.couder@xxxxxxxxx> wrote:
> > If you take a look at c6f44e1da5 ("t9813: avoid using pipes",
> > 2017-01-04) you can see the following:
> > - it changes only one test file: t9813-git-p4-preserve-users.sh
> > - its title starts with "t9813: "
> I adapted this format in my second version.
> : https://public-inbox.org/git/20190310080739.63984-1-ttjtftx@xxxxxxxxx
It's better because each patch changes only one file, but I also
wanted to say that for a microproject you only need to focus on only
one file. So I would you suggest you keep only the patches that are
about "t0000-basic.sh" and drop the patches about
"t0003-attributes.sh" and "t0022-crlf-rename.sh".
(https://git.github.io/SoC-2019-Microprojects/ has now a "Only ONE
quality focused microproject per student" section with more content.)
It's also a small nit but your patches start with "t0000-basic: " not
just "t0000: " though the latter format is more frequent in existing
commits than the former.