<p><b>@codebrainz</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/1806#discussion_r179649195">src/keybindings.h</a>:</p>
<pre style='color:#555'>> @@ -274,6 +274,8 @@ enum GeanyKeyBindingID
        GEANY_KEYS_FORMAT_SENDTOCMD8,                           /**< Keybinding. */
        GEANY_KEYS_FORMAT_SENDTOCMD9,                           /**< Keybinding. */
        GEANY_KEYS_EDITOR_DELETELINETOBEGINNING,        /**< Keybinding. */
+       GEANY_KEYS_DOCUMENT_STRIPTRAILINGSPACES,        /**< Keybinding. 
+                                                                                                * @since 1.34   */
</pre>
<p>I like the current practice of putting both the Geany version and the API version since as a plugin developer you're forced to know both and not have to reverse engineer one from the other is useful (which is why this was started IIRC). You need to know the Geany version to know which releases of Geany the plugin is actually compatible with, but you also need to know the API version to satisfy the <code>plugin_version_check()</code> thing in the plugin API.</p>
<p>In a perfect world we'd just get rid of the plugin API version and use only the release version to make the whole concept simpler. I have <a href="https://github.com/codebrainz/geany/commit/2a678cf0513fe7fd5163708c586adf5c545ed589">an old commit</a> that added this in a backwards compatible way. IIRC there was an objection because of the theoretical case where some developer might want to use intermediate API versions in-between release without having to keep their feature branch up-to-date or something.</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/1806#discussion_r179649195">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ4IaKYQfBkqiY8t7YGVsrUV4hY-7ks5tltcmgaJpZM4Sq9cc">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJyOnideigZPP5lSjtAnNg468y-x9ks5tltcmgaJpZM4Sq9cc.gif" height="1" width="1" alt="" /></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/1806#discussion_r179649195"></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":"@codebrainz commented on #1806"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1806#discussion_r179649195"}}}</script>