<p>Its not a hard and fast rule (Geany has relatively few of them), but if something gets to the point where its actually usable, then doing further development on a pull request is occasionally done, the problem is that anyone contributing more than just comments has to make the pull request to your repo anyway, and pull requests there are not visible on the Geany one until its merged, resulting in two places to be tracked.  Thats a failing of github.</p>
<p>But irrespective of the development method, making all changes in a branch is mandatory, nobody will test something if it changes their master, master should purely track upstream, ie the Geany github repo.  It also helps you too, you can sidetrack your changes simply by checking out a new branch, or back to master to pull down any changes from upstream.</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/1687#issuecomment-344119038">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJy7guFhrypqBgKXf6kETRu3MoTdTks5s2PEigaJpZM4Qcov5">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ1cvkgzvRk4qL8QrIlwBaBleMwueks5s2PEigaJpZM4Qcov5.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/1687#issuecomment-344119038"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<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://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@elextr in #1687: Its not a hard and fast rule (Geany has relatively few of them), but if something gets to the point where its actually usable, then doing further development on a pull request is occasionally done, the problem is that anyone contributing more than just comments has to make the pull request to your repo anyway, and pull requests there are not visible on the Geany one until its merged, resulting in two places to be tracked.  Thats a failing of github.\r\n\r\nBut irrespective of the development method, making all changes in a branch is mandatory, nobody will test something if it changes their master, master should purely track upstream, ie the Geany github repo.  It also helps you too, you can sidetrack your changes simply by checking out a new branch, or back to master to pull down any changes from upstream."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1687#issuecomment-344119038"}}}</script>