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