D16215: Cleanup
Dominik Haumann
noreply at phabricator.kde.org
Mon Oct 15 20:32:06 BST 2018
dhaumann accepted this revision.
dhaumann added a comment.
This revision is now accepted and ready to land.
The title "Cleanup" is a bit too generic imho. Better eould be something along the lines "Ceanup: Merge main() into unit test tsttestapp.cpp" along with a detailed description :) Maybe something for future commits? Good enough this time, but for commits in KDE Frameworks this would not be good enough, since "Cleanup" would then end up in David's changelog, which is too generic (cleanup of what?).
Please push.
REPOSITORY
R40 Kate
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D16215
To: gregormi, #kate, dhaumann
Cc: dhaumann, kwrite-devel, michaelh, ngraham, demsking, cullmann, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20181015/52ea0a33/attachment.html>
More information about the KWrite-Devel
mailing list