D8830: Register kdevelop as handler for repo urls, add --fetch argument

Kevin Funk noreply at phabricator.kde.org
Thu Nov 16 08:52:58 UTC 2017


kfunk accepted this revision.
kfunk added a comment.


  Nice idea indeed!

INLINE COMMENTS

> projectcontroller.cpp:859
> +
> +    for (auto* plugin : vcsPlugins) {
> +        auto* iface = plugin->extension<IBasicVersionControl>();

Could be factored out into a `vcsProviderPluginForUrl(...)` method.

REPOSITORY
  R32 KDevelop

BRANCH
  registerkdevelopasrepourlhandler

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

To: kossebau, #kdevelop, apol, kfunk
Cc: kfunk, apol, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20171116/d99281cf/attachment.html>


More information about the KDevelop-devel mailing list