<p>The reason you are getting all the updates and all other changes you make is that you have made the pull request from master not from a branch.  If you make a branch there is not need to touch it once you have made the pull request, (except if changes are needed :) and so the PR doesn't see all the noise or accidently get other changes you make (especially as you will make those in other branches now you know about them :) and your master will be clean and able to pull upstream changes without problems.  It also makes it easier for testers to not pollute their master with your changes.</p>
<p>Sorry should have pinged you on that first but I didn't notice it wasn't a branch.</p>
<p>On the PR itself.</p>
<p>We don't really want to add more trash to the various prefs, but on the other hand adding it to the proper prefs UI needs Glade 3.8.5 (<a class="user-mention" data-hovercard-user-id="5799394" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/Kugel">@Kugel</a> does 3.8.6 work?) and on any reasonably recent distro that version is not in the package repository so you have to build it yourself from source.</p>
<p>Otherwise it looks ok by quick inspection, not tested.</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/1908#issuecomment-416872261">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ6Z3uGq99VnIqgIlXNzVIc7p97uCks5uVlKMgaJpZM4VjSJn">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJ-U6zBJtyEvYtYD9KB0d6hm7kPbsks5uVlKMgaJpZM4VjSJn.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://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.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 #1908: The reason you are getting all the updates and all other changes you make is that you have made the pull request from master not from a branch.  If you make a branch there is not need to touch it once you have made the pull request, (except if changes are needed :) and so the PR doesn't see all the noise or accidently get other changes you make (especially as you will make those in other branches now you know about them :) and your master will be clean and able to pull upstream changes without problems.  It also makes it easier for testers to not pollute their master with your changes.\r\n\r\nSorry should have pinged you on that first but I didn't notice it wasn't a branch.\r\n\r\nOn the PR itself.\r\n\r\nWe don't really want to add more trash to the various prefs, but on the other hand adding it to the proper prefs UI needs Glade 3.8.5 (@kugel does 3.8.6 work?) and on any reasonably recent distro that version is not in the package repository so you have to build it yourself from source.\r\n\r\nOtherwise it looks ok by quick inspection, not tested."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1908#issuecomment-416872261"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/1908#issuecomment-416872261",
"url": "https://github.com/geany/geany/pull/1908#issuecomment-416872261",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [geany/geany]  #1907 Add preferences flag to skip the project_ask_close dialog  (#1908)",
"sections": [
{
"text": "",
"activityTitle": "**elextr**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@elextr",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"geany/geany\",\n\"issueId\": 1908,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close pull request",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"PullRequestClose\",\n\"repositoryFullName\": \"geany/geany\",\n\"pullRequestId\": 1908\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/geany/geany/pull/1908#issuecomment-416872261"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 361570919\n}"
}
],
"themeColor": "26292E"
}
]</script>