Re: [PATCH v2 1/2] conf: Add <lease/> option for <dhcp/> settings
On 4/22/20 9:22 AM, Julio Faracco wrote:
Hi Daniel,
Thanks for reviewing. :-)
Np, glad to be of assistance
IMHO, I don't like to join them in one single patch because it is
hard
to review.
Hmm, reading this I believe that the tests would pass after applying patch
2/2 then (didn't try).
I know that others have a different opinion (and it is good).
The way you split the patch series is up to debate and so on and so
forth. The issue here is that every single patch must pass 'make check',
as said in docs/hacking.rst in the "Preparing patches" session:
----------
If you're going to submit multiple patches, the automated tests
must pass **after each patch**, not just after the last one.
----------
You'll need to think in a way of splitting the series that allows each
patch to pass 'make check' in its own.
Thanks,
DHB