<table><tr><td style="">meven added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D26912">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D26912#603041" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D26912#603041</a>, <a href="https://phabricator.kde.org/p/alex/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@alex</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Hello,</p>

<p>I already wondered how this will be handled.</p>

<p>Should I abandon this revision now and maybe create a new one with the relevant changes once your request has been merged ?</p></div>
</blockquote>

<p>I think that's the easiest way. Hopefully you use a script for this formatting and won't loose too much time.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>And is there any way I could have known that this kcm is going to be refactored ?</p></blockquote>

<p>Unfortunately there is no easy way but follow a little the activity on phabricator which is how I found your review.</p>

<p>I wish we had some notification/information when we touch a file or folder that has currently running reviews.<br />
I don't know any forge platform that does this kind of thing.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>And a general question: When are accepted patches merged ?</p></blockquote>

<p>It is variable.<br />
The smallest the more inconsequential and more consensual, the more tested and the more easy to test, the more people and experienced people have accepted it, the fastest it should land.<br />
On the opposite, the more code you touch or on the more sensible things should make you should be patient and give reasonable time for other reviewers to comment/accept/or require change, especially experienced reviewers or maintainers.<br />
You will then probably need to remind reviewers to have a look. We have sortof a reviewer shortage.<br />
Doc does not specify this :/ <a href="https://community.kde.org/Infrastructure/Phabricator#Step_3:_Land_your_diff" class="remarkup-link" target="_blank" rel="noreferrer">https://community.kde.org/Infrastructure/Phabricator#Step_3:_Land_your_diff</a></p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>For instance your patch <a href="https://phabricator.kde.org/D26478" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D26478</a> is very simple and has been accepted for a few weeks now, but has not been merged.</p></blockquote>

<p>This one I forgot about, I thought I landed it.</p>

<p>A tip, use <a href="https://phabricator.kde.org/differential/query/active/" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/differential/query/active/</a> to have a quick overview on your reviews.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26912">https://phabricator.kde.org/D26912</a></div></div><br /><div><strong>To: </strong>alex, mlaurent, davidedmundson, meven<br /><strong>Cc: </strong>meven, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>