> On Dec. 16, 2016, 2:55 a.m., Stephen Kelly wrote:
> > Thanks for working on this!
> >
> > I noticed that I could remove the change in beginRemoveRows without any
> > unit test failing.
> >
> > Please add this test on top of your patch:
> >
> > diff --git a/autotests/kselectionproxymodeltes
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129657/
---
(Updated Dec. 16, 2016, 9:23 a.m.)
Status
--
This change has been ma
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129657/#review101464
---
Ship it!
Thanks for working on this!
I noticed that I co
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129657/#review101463
---
Ship it!
Ship It!
- Stephen Kelly
On Dec. 15, 2016, 11
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129657/#review101460
---
+1 looks good.
- Aleix Pol Gonzalez
On Dec. 16, 2016, 12:
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129657/
---
Review request for KDE Frameworks and Stephen Kelly.
Repository: kitemmod