[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] t/pm: Use Test::Simple for Perl tests
From: |
Matthias Paulmier |
Subject: |
Re: [PATCH] t/pm: Use Test::Simple for Perl tests |
Date: |
Thu, 06 Sep 2018 08:58:50 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Mathieu Lirzin <address@hidden> writes:
> Hello Matthias,
>
> Matthias Paulmier <address@hidden> writes:
>
>> [...]
>
> This patch is too fat for me to properly review it. The changelog
> description makes me think it could easily be separated in smaller
> chunks. For example you could make one patch adding support for
> ‘Test::Simple’ and a basic test file with a ‘.plt’ extension, one patch
> for each test file migrated to Test::Simple, one for fixing the XFAIL
> hack...
>
> Feel free to adapt to your particular case, since you understand the
> dependencies between the change you made better than I do.
>
> Can you send the updated patches?
I sure can, I was feeling like it was a bit long but for some reason
thought it was better that way.
Can I send multiple patches at once (in one email as attachements), or
should I send them each on their own?
--
Matthias Paulmier