<div dir="ltr">@Andreas<div>It's a bit hidden: You need to click on "Merge pull-request", then do <b>not</b> click on "Confirm merge", but on the small arrow to the right, and select "Squash and merge".</div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-06-14 18:13 GMT+02:00 Andreas Mueller <span dir="ltr"><<a href="mailto:t3kcit@gmail.com" target="_blank">t3kcit@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    I'm +1 for using the button when appropriate.<br>
    I think it should be up to the merging person to make a call whether
    a squash is a better<br>
    logical unit than all the commits.<br>
    I would set like a soft limit at ~5 commits or something. If your PR
    has more than 5 separate<br>
    big logical units, it's probably too big.<br>
    <br>
    The button is enabled in the settings but I can't see it.<br>
    Am I being stupid?<div><div class="h5"><br>
    <br>
    <div>On 06/14/2016 06:58 AM, Joel Nothman
      wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr">Sounds good to me. Thank goodness someone reads the
        documentation!</div>
      <div class="gmail_extra"><br>
        <div class="gmail_quote">On 14 June 2016 at 19:51, Alexandre
          Gramfort <span dir="ltr"><<a href="mailto:alexandre.gramfort@telecom-paristech.fr" target="_blank">alexandre.gramfort@telecom-paristech.fr</a>></span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>> We could stop squashing during development,
              and use the new Squash-and-Merge<br>
              > button on GitHub.<br>
              > What do you think?<br>
              <br>
            </span>+1<br>
            <br>
            the reason I see for squashing during dev is to avoid
            killing the<br>
            browser when reviewing. It really rarely happens though.<br>
            <span><font color="#888888"><br>
                A<br>
              </font></span>
            <div>
              <div>_______________________________________________<br>
                scikit-learn mailing list<br>
                <a href="mailto:scikit-learn@python.org" target="_blank">scikit-learn@python.org</a><br>
                <a href="https://mail.python.org/mailman/listinfo/scikit-learn" rel="noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/scikit-learn</a><br>
              </div>
            </div>
          </blockquote>
        </div>
        <br>
      </div>
      <br>
      <fieldset></fieldset>
      <br>
      <pre>_______________________________________________
scikit-learn mailing list
<a href="mailto:scikit-learn@python.org" target="_blank">scikit-learn@python.org</a>
<a href="https://mail.python.org/mailman/listinfo/scikit-learn" target="_blank">https://mail.python.org/mailman/listinfo/scikit-learn</a>
</pre>
    </blockquote>
    <br>
  </div></div></div>

<br>_______________________________________________<br>
scikit-learn mailing list<br>
<a href="mailto:scikit-learn@python.org">scikit-learn@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/scikit-learn" rel="noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/scikit-learn</a><br>
<br></blockquote></div><br></div>