[Differential] [Updated] D932: Cannot find the 'MERGE' option into KDevPlatform VCS support source code

[mwolff (Milian Wolff)]

mwolff added a comment.


  +1 in general, but I also would like to see a unit test for this

INLINE COMMENTS
  plugins/openwith/kdevopenwith.json:23 unrelated
  vcs/dvcs/ui/branchmanager.cpp:86 please set an icon, I have a "merge" icon, so use that?
  vcs/interfaces/ibranchingversioncontrol.h:102 wrong comment,no?
  vcs/interfaces/ibranchingversioncontrol.h:105 add newline after this line

REPOSITORY
  rKDEVPLATFORM KDevPlatform

REVISION DETAIL
  https://phabricator.kde.org/D932

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: coliveira, apol, kfunk, mwolff
Cc: kfunk, kdevelop-devel, arrowdodger