<div dir="ltr">Hello Robert,<div><br></div><div><br></div><div>If you have clearly defined feature branches, then it's probably better to make one PR for each, for reviewing purposes.</div><div><br></div><div><br></div>
<div>Raphaël</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Feb 23, 2014 at 11:35 AM, Robert Drake <span dir="ltr"><<a href="mailto:rdrake@direcpath.com" target="_blank">rdrake@direcpath.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF">
    Hello again,<br>
    <br>
    I think I'm done with my changes for now.  I can parse my giant
    dhcpd.conf file so I'm happy.  I've got everything broken into
    features so it's 12 minor patches right now.  Would you guys rather
    have one commit with everything rolled into it, or keep it separate?<br>
    <br>
    Thanks,<br>
    Robert<div class=""><br>
    <br>
    <div>On 2/23/2014 2:35 AM, Raphaël Pinson
      wrote:<br>
    </div>
    <blockquote type="cite">
      
      <p dir="ltr">Hello Robert, </p>
      <p dir="ltr">This would indeed be an incompatible change, but a
        kind we've introduced for other lenses in the past, so it's
        acceptable imo.</p>
      <p dir="ltr">As for how to fix this technically, the Quote module
        should now be used for this. </p>
      <p dir="ltr">Regards, </p>
      <p dir="ltr">--<br>
        Raphaël Pinson<br>
        Infrastructure Developer & Trainer<br>
        <a href="tel:%2B33%20479%2026%2057%2093" value="+33479265793" target="_blank">+33 479 26 57 93</a><br>
        <a href="tel:%2B33%20781%2090%2000%2079" value="+33781900079" target="_blank">+33 781 90 00 79</a></p>
      <p dir="ltr">Camptocamp France<br>
        Savoie Technolac<br>
        BP 352<br>
        48, avenue du Lac du Bourget<br>
        73372 Le Bourget-du-Lac, Cedex</p>
    </blockquote>
    <br>
  </div></div>

</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Raphaël Pinson<div>Infrastructure Developer & Trainer</div><div>+33 479 26 57 93<div>+33 781 90 00 79</div><div><br></div><div>Camptocamp France<br>

Savoie Technolac<br>
BP 352<br>
48, avenue du Lac du Bourget<br>
73372 Le Bourget du Lac, Cedex<br>
<a href="http://www.camptocamp.com" target="_blank">www.camptocamp.com</a><br>
</div></div></div>
</div>