<p></p>
<p dir="auto">I am not really interested in this PR in its current regressed state.</p>
<blockquote>
<p dir="auto">It's your choice, we can just take the current state and merge it (after some squashing) and deal headers next or you get this in with <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/elextr/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/elextr">@elextr</a> who was pretty happy with the earlier state. I'm not going to engage in further discussions if the header patch should be part of this PR or not, my time is better spend reviewing actual patches.</p>
</blockquote>
<p dir="auto"><a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/kugel-/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/kugel-">@kugel-</a> so your approach is to smash and run.  You demand major changes just before merge so the PR is no longer in the state where I was pretty happy with it, completely wasting my previous work of test and inspect. And now you want to cry time poor when asked to provide discussion of the change you demanded.  Everybody is time poor, none of us are full time Geanyists, so forcing extra work on people without contributing simply makes you rude and damaging to the project.  I am also still waiting on replies to my comments on your own PRs <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="1213640861" data-permission-text="Title is private" data-url="https://github.com/geany/geany/issues/3178" data-hovercard-type="pull_request" data-hovercard-url="/geany/geany/pull/3178/hovercard" href="https://github.com/geany/geany/pull/3178">#3178</a> and <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="306929795" data-permission-text="Title is private" data-url="https://github.com/geany/geany/issues/1813" data-hovercard-type="pull_request" data-hovercard-url="/geany/geany/pull/1813/hovercard" href="https://github.com/geany/geany/pull/1813">#1813</a>, clearly other peoples contributions don't matter to you.</p>
<p dir="auto"><a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/techee/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/techee">@techee</a> as I said before I am disappointed that you chose to regress this PR before discussion with someone who is helping you, so wasting their efforts.  <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/kugel-/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/kugel-">@kugel-</a> is right in part, significant changes should be discussed on issues before PRs are made, then multiple partial PRs make sense and they can be smaller without large numbers of commits that need complex squashing at merge time, they should be able to be squashed flat at merge (which can't introduce bugs).  But big changes mid PR is a waste of everybodys efforts, do not do that without discussing it with them.</p>
<p dir="auto">To you both, as I have said before, I do not use Geany for C++, the state of this PR does not help me, I was just trying to help <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/techee/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/techee">@techee</a> provide as useful a user experience for those who do so.   So completely disregarding that effort by demanding and making wholesale changes to the PR without any discussion has left me unhappy about engaging any further with this PR.</p>
<p dir="auto"><a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/kugel-/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/kugel-">@kugel-</a> correctly pointed out that there is a lack of effort available to the project ATM and incidents like this don't help, as he correctly said they suck up effort.  More discussion and planning is needed, and it does not help that contributors are in different time zones, and I appreciate the extra efforts of those for whom English is not their first language, but it is what it is.</p>
<blockquote>
<p dir="auto">Ultimately, I'm even OK with merging PRs without review if it's really impossible to get any feedback (not uncommon these days, unfortunately) though it come with a kind of warning in advance.</p>
</blockquote>
<p dir="auto">As I noted above you have not replied to feedback on several PRs, however I agree with the sentiment, the process for simple bug fixes should be simple, an OP should be allowed to merge their own simple PRs providing warning and sufficient time is allowed (I suggest two weeks, including two weekends) if no feedback is received.  But additional features, or changes to existing features really need to be discussed more, not less, to avoid more of this sort of occurrence.  One thing I definitely agree with <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/kugel-/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/kugel-">@kugel-</a> is that discussions over code is not the best way to do it.</p>
<p dir="auto">So how to progress this PR now.</p>
<p dir="auto">As <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/kugel-/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/kugel-">@kugel-</a> has said he is not blocking it whichever way, and I am no longer interested in pushing it, so that leaves <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/techee/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/techee">@techee</a> to decide.  Which is fair enough <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/techee/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/techee">@techee</a> has done all the work.</p>
<p dir="auto">I guess <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/kugel-/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/kugel-">@kugel-</a> can't build and test on the train so if <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/techee/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/techee">@techee</a> gets the PR into an immediately mergable state, to get progress I will do a quick scan, and build and test and if ok press the green button.  But <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/techee/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/techee">@techee</a> whichever state you decide on, please document the intended behaviour so I know what to test.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/3185#issuecomment-1121769338">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ2MMDO7SPKTLZNFED3VJG3UBANCNFSM5UWGU33A">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAIOWJ53FXHUPZEXMR3BJLTVJG3UBA5CNFSM5UWGU33KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOILONO6Q.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><geany/geany/pull/3185/c1121769338</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/3185#issuecomment-1121769338",
"url": "https://github.com/geany/geany/pull/3185#issuecomment-1121769338",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>