<table><tr><td style="">bruns 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/D16579">View Revision</a></tr></table><br /><div><div><p>The change, as you pushed it, was never open for review, so there was not even the chance to review.</p>

<p>The new test files change the tested tag from one allowed variant to another. Both variants exist in the wild, and you changed the test coverage to the one you seemed to be more important. A good change would have increased the coverage.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R286 KFileMetaData</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16579">https://phabricator.kde.org/D16579</a></div></div><br /><div><strong>To: </strong>smithjd, astippich, bruns, mgallien<br /><strong>Cc: </strong>ngraham, bruns, astippich, kde-frameworks-devel, Baloo, gennad, domson, ashaposhnikov, michaelh, spoorun, abrahams<br /></div>