<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=1010248" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/frlan">@frlan</a> yes, its good rule at the time of <em>merging</em> a pull request, but squashing commits along the way leads to force-pushing which can cause problems.  So its not something  <em>contributors</em> should do until the PR is accepted, but ...</p>
<p>... (as you said) github now provides good support squashing at merge (its the default big green button) so its even less important now.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/2059#issuecomment-455508460">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ1WMcGeATAr9Ri7AJKkAfKR6UR4Dks5vEakYgaJpZM4aHHkR">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJwqn6wRO4J_bjVXH1Pti_yZSmHG6ks5vEakYgaJpZM4aHHkR.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany","title":"geany/geany","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@elextr in #2059: @frlan yes, its good rule at the time of _merging_ a pull request, but squashing commits along the way leads to force-pushing which can cause problems.  So its not something  _contributors_ should do until the PR is accepted, but ...\r\n\r\n... (as you said) github now provides good support squashing at merge (its the default big green button) so its even less important now.\r\n"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/2059#issuecomment-455508460"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2059#issuecomment-455508460",
"url": "https://github.com/geany/geany/pull/2059#issuecomment-455508460",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>