<blockquote>
<p>He said he didn't feel strongly, and obviously I implemented it this way so I at least feel more strongly. But point taken.</p>
</blockquote>

<p>I'm not really saying either way is better per se, but you said you didn't care much, and he asked you to change it (<a href="https://github.com/geany/geany/pull/1212#issuecomment-245685550" class="issue-link js-issue-link" data-url="https://github.com/geany/geany/issues/1212" data-id="175140331" data-error-text="Failed to load issue title" data-permission-text="Issue title is private">#1212 (comment)</a>).  As the discussion reads from here, it <em>looks</em> like you ignored it.<br>
Same with showing the plugin name in the warning (<a href="https://github.com/geany/geany/pull/1212#r77622584" class="issue-link js-issue-link" data-url="https://github.com/geany/geany/issues/1212" data-id="175140331" data-error-text="Failed to load issue title" data-permission-text="Issue title is private">#1212 (comment)</a>) -- probably a mere forgetfulness, though.</p>

<p>I'm not trying to blame or try and stop people from going ahead with merges, just trying to smooth things out and get everyone happy :)</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/1212#issuecomment-247564169">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ6zHnyt5OjOCCpCQ-P0rbJfwuLmUks5qqmmEgaJpZM4J1X_v">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ86f4daeGVV-PDPiLRkNnSy-XO_yks5qqmmEgaJpZM4J1X_v.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/1212#issuecomment-247564169"></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":"@b4n in #1212: \u003e He said he didn't feel strongly, and obviously I implemented it this way so I at least feel more strongly. But point taken.\r\n\r\nI'm not really saying either way is better per se, but you said you didn't care much, and he asked you to change it (https://github.com/geany/geany/pull/1212#issuecomment-245685550).  As the discussion reads from here, it *looks* like you ignored it.\r\nSame with showing the plugin name in the warning (https://github.com/geany/geany/pull/1212#r77622584) -- probably a mere forgetfulness, though.\r\n\r\nI'm not trying to blame or try and stop people from going ahead with merges, just trying to smooth things out and get everyone happy :)"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1212#issuecomment-247564169"}}}</script>