<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=811085" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/elextr">@elextr</a> please don't squash-merge this kind of big PRs with great commit splitting, we now lost the whole history here, and although yes, some commits were big already, most were very well split apart.  I'll also make bisecting a potential issue a lot harder.</p>
<p>Anyway, as per the merge itself: I obviously am not a fan, but well, I also admit we need to move forward at some point, and this kind of forces (my) hand.  Sad it happens right when I'm back with some time for Geany dev… well I guess them I should have <em>started</em> with this one yesterday instead of <a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="414483109" data-permission-text="Issue title is private" data-url="https://github.com/geany/geany/issues/2092" data-hovercard-type="issue" data-hovercard-url="/geany/geany/issues/2092/hovercard" href="https://github.com/geany/geany/issues/2092">#2092</a>.</p>
<blockquote>
<ol>
<li>it fixes three languages, Asciidoc, Rest (Geanys own documentation) and Latex and progresses the process of making it easier to maintain parsers from upstream.</li>
</ol>
</blockquote>
<p>Let's just be fair: I already proposed a standalone fix for Asciidoc and Rest (<a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="392036866" data-permission-text="Issue title is private" data-url="https://github.com/geany/geany/issues/2019" data-hovercard-type="pull_request" data-hovercard-url="/geany/geany/pull/2019/hovercard" href="https://github.com/geany/geany/pull/2019">#2019</a>) without this PR, and you rejected it because merging this was a better choice -- which is fair enough, but then not so much an argument for pushing this one :)<br>
I wasn't aware that Latex was broken in master, probably to blame on my lack of Geany time lately.</p>
<blockquote>
<ol start="2">
<li>It has been working for me for months</li>
</ol>
</blockquote>
<p>That's a great point :)</p>
<blockquote>
<ol start="3">
<li>If we keep delaying every ctags update for a year we will always have huge updates :(</li>
</ol>
</blockquote>
<p>yes, but with this PR, so long as it can still be merged in Geany, we should have a clean-ish diff from uctags, meaning update should be fairly easy no matter how big they are, as most can just be applied as-is.</p>
<blockquote>
<p>The downside is it loses Cobol</p>
</blockquote>
<p>An ActionScript as well.  I don't think we can release in the current state without those, especially as we have at least one active COBOL user that even submitted changes recently.</p>
<p>Anyway, we now have to pull up our sleeves and fix what needs to be fixed before the release.  Help is welcome.</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/2018#issuecomment-480483101">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ1QeJfEftxRM_zUaRtAoKPXN_vgaks5veE-4gaJpZM4ZXOnd">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJ4PHatCYwwb13PN1Z3BZ67pA76bIks5veE-4gaJpZM4ZXOnd.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":"@b4n in #2018: @elextr please don't squash-merge this kind of big PRs with great commit splitting, we now lost the whole history here, and although yes, some commits were big already, most were very well split apart.  I'll also make bisecting a potential issue a lot harder.\r\n\r\nAnyway, as per the merge itself: I obviously am not a fan, but well, I also admit we need to move forward at some point, and this kind of forces (my) hand.  Sad it happens right when I'm back with some time for Geany dev… well I guess them I should have *started* with this one yesterday instead of #2092.\r\n\r\n\u003e 1. it fixes three languages, Asciidoc, Rest (Geanys own documentation) and Latex and progresses the process of making it easier to maintain parsers from upstream.\r\n\r\nLet's just be fair: I already proposed a standalone fix for Asciidoc and Rest (#2019) without this PR, and you rejected it because merging this was a better choice -- which is fair enough, but then not so much an argument for pushing this one :)\r\nI wasn't aware that Latex was broken in master, probably to blame on my lack of Geany time lately.\r\n\r\n\u003e 2. It has been working for me for months\r\n\r\nThat's a great point :)\r\n\r\n\u003e 3. If we keep delaying every ctags update for a year we will always have huge updates :(\r\n\r\nyes, but with this PR, so long as it can still be merged in Geany, we should have a clean-ish diff from uctags, meaning update should be fairly easy no matter how big they are, as most can just be applied as-is.\r\n\r\n\u003e The downside is it loses Cobol\r\n\r\nAn ActionScript as well.  I don't think we can release in the current state without those, especially as we have at least one active COBOL user that even submitted changes recently.\r\n\r\nAnyway, we now have to pull up our sleeves and fix what needs to be fixed before the release.  Help is welcome."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/2018#issuecomment-480483101"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2018#issuecomment-480483101",
"url": "https://github.com/geany/geany/pull/2018#issuecomment-480483101",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>