D16579: Remove support for non-standard APE tag field names from the test files
James Smith
noreply at phabricator.kde.org
Mon Mar 11 05:06:50 GMT 2019
smithjd added a comment.
In D16579#428790 <https://phabricator.kde.org/D16579#428790>, @ngraham wrote:
> Two reviewers had open "Changes Requested" statuses on this patch when you committed it, and from the latest comments, it's not clear that their concerns were addressed first. Can you explain your thought process regarding why you felt it was appropriate to ignore your reviewers as well as KDE community norms and commit this anyway?
The scope has shrunk to just the test files since D18826 <https://phabricator.kde.org/D18826> made the original code changes and the requested changes obsolete, and the test file changes were ack'ed.
Your comment is unfounded.
REPOSITORY
R286 KFileMetaData
REVISION DETAIL
https://phabricator.kde.org/D16579
To: smithjd, astippich, bruns, mgallien
Cc: ngraham, bruns, astippich, kde-frameworks-devel, #baloo, gennad, domson, ashaposhnikov, michaelh, spoorun, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190311/f6ffa08b/attachment.html>
More information about the Kde-frameworks-devel
mailing list