<table><tr><td style="">ltoscano reopened this revision.<br />ltoscano added a comment.<br />This revision is now accepted and ready to land.
</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/D7928" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Two comments (sorry, I'm late):</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">If the patch was mostly the one from comment <a href="https://bugs.kde.org/show_bug.cgi?id=350731#c39" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=350731#c39</a>, it should be committed with the wrong author (the person who sent it); if it's the case, please revert and commit again after fixing the author (git commit --author).</li>
<li class="remarkup-list-item">it should probably go to Applications/17.08</li>
<li class="remarkup-list-item">in general, please try to use arcanist; there is a disconnection between what was written here (and was part of the evaluation) and what ended in the commit message.</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R201 Akregator</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7928" rel="noreferrer">https://phabricator.kde.org/D7928</a></div></div><br /><div><strong>To: </strong>marten, KDE PIM, dvratil<br /><strong>Cc: </strong>ltoscano, anthonyfieroni, cfeck, dvasin, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>