Review Request 125533: dragon-plugins functionality patch

Emmanuel Pescosta emmanuelpescosta099 at gmail.com
Wed Oct 7 08:00:37 BST 2015



> On Oct. 6, 2015, 2:32 p.m., Anthony Fieroni wrote:
> > Parsing remotes/, example remotes/origin/Applications/15.08, is not correct.
> 
> Anthony Fieroni wrote:
>     Misclick, this was comment of patch1.

https://git.reviewboard.kde.org/r/new/ <- Open new review requests here

Please open a new review request FOR EACH patch file 
instead of attaching all patches to one review request, 
so that we can review your changes via [View Diff]. 

So in the end there should be 4 review requests from you: One with patch1, one with patch2, ...

Thanks! :)


- Emmanuel


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125533/#review86419
-----------------------------------------------------------


On Oct. 6, 2015, 6:41 a.m., Anthony Fieroni wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125533/
> -----------------------------------------------------------
> 
> (Updated Oct. 6, 2015, 6:41 a.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Repository: dolphin-plugins
> 
> 
> Description
> -------
> 
> Discussion about functionality, patchs under review
> 
> 
> Diffs
> -----
> 
>   git/pulldialog.cpp 9219e82 
>   git/pushdialog.cpp 5c36932 
> 
> Diff: https://git.reviewboard.kde.org/r/125533/diff/
> 
> 
> Testing
> -------
> 
> One by one patch review, same as their name.
> 1. Correct parsing remotes/, example remotes/origin/Applications/15.08
> 2. Enable commit on modified non confict items, enable show differences on modified or conflict items
> 3. Correct mark modified item, example test/ must not be highlighted when modification are in test1/, match exacte name not only part of them
> 4. Revert on svn files
> ---------------------------
> Discussions:
> 1. Logs, it are needed
> 2. Checkout on single/selected files/folders, git plugin
> 3. git commit -a + push on git/commitdialog
> 
> 
> File Attachments
> ----------------
> 
> patch1
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/10/05/91adf078-70a0-4c9d-a16b-c4ad9cfd4fc5__patch1
> patch2
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/10/05/b2c2a205-711f-4171-98f6-75ca8094cf61__patch2
> patch3
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/10/05/89f7b16d-6fb1-49d9-a682-1e768afc2f6c__patch3
> patch4
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/10/05/a83fd3e3-f6bc-49a1-96e5-abf5d6968495__patch4
> 
> 
> Thanks,
> 
> Anthony Fieroni
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20151007/ebd83eaf/attachment.htm>


More information about the kfm-devel mailing list