Updates in response to KVersionControlPlugin interface changes.
ClosedPublic

Authored by nikolaik on Apr 21 2020, 9:24 AM.

Details

Summary

Updates VCS-plugins for the new KVersionControlPlugin interface. Updates available in the separate commit D29041.
Plugins unversionedActions() functions now do nothing so it doesn't change any current plugin behaviour.

Diff Detail

Repository
R449 Plugins for Dolphin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
nikolaik requested review of this revision.Apr 21 2020, 9:24 AM
nikolaik created this revision.
meven requested changes to this revision.EditedApr 21 2020, 10:30 AM

Git, mercurial and bazar can do things to unversionned files : add them to their index.
I am not sure if that is a regression though or if you can still add files.

Not sure about dropbox.

This revision now requires changes to proceed.Apr 21 2020, 10:30 AM
meven accepted this revision as: meven.Apr 21 2020, 10:39 AM

Git, mercurial and bazar can do things to unversionned files : add them to their index.
I am not sure if that is a regression though or if you can still add files.

Not sure about dropbox.

Apparently not, unversionedActions is an unfortunate name for those plugins as unversionned has a different meaning and got me confused.

meven accepted this revision.Apr 21 2020, 10:39 AM
This revision is now accepted and ready to land.Apr 21 2020, 10:39 AM

Please don't "BUG: 420239" as it is not actually fixing it yet and assume the checkout/clone actions will come in a subsequent differential revision.
And use "FEATURE: 420239" since this will be a new feature.

nikolaik edited the summary of this revision. (Show Details)Apr 21 2020, 12:31 PM
nikolaik updated this revision to Diff 80769.Apr 21 2020, 2:39 PM
  • Name changed: unversionedActions() -> outOfVersionControlActions().
nikolaik updated this revision to Diff 80789.Apr 21 2020, 4:47 PM
  • Name change: versionedActions() -> versionControlActions()
meven edited the summary of this revision. (Show Details)Apr 22 2020, 7:58 AM
meven accepted this revision.Apr 22 2020, 8:00 AM
meven edited the summary of this revision. (Show Details)

Your changes are all good ;)

But you can't FEATURE: 420239 since the feature is not implemented here, (at least as of now).
Please run arc amend on your local branch to remove it from the commit message.

elvisangelaccio accepted this revision.Apr 26 2020, 5:19 PM
elvisangelaccio added inline comments.
bazaar/fileviewbazaarplugin.cpp
277

Semicolon not necessary. Same for all other Q_UNUSED below.

nikolaik updated this revision to Diff 81280.Apr 26 2020, 8:00 PM
  • Update in response to review comments.
nikolaik marked an inline comment as done.Apr 26 2020, 8:00 PM
meven accepted this revision.Apr 26 2020, 9:12 PM

ping
can i push this?

ping
can i push this?

Yes ;)

When @elvisangelaccio approved with a small comment, he meant fix those small things and GTM.

Can't wait to see the checkout/clone here actions ;)