PyKF5 bindings generator improvements out for review.

Shaheed Haque srhaque at theiet.org
Fri Feb 3 21:53:55 UTC 2017


Steve,

PRs 2 through 7, at
https://github.com/ShaheedHaque/extra-cmake-modules/pulls are now the
complete set of the changes required to bring the mainstream'd fork up to
par with mine.

I'll be continuing the bringup of the automatic QMap/QList/etc and array
stuff; hopefully you'll have a chance to review the above.

Thanks, Shaheed


On 31 January 2017 at 22:26, Shaheed Haque <srhaque at theiet.org> wrote:

> And now https://github.com/ShaheedHaque/extra-cmake-modules/pull/4...
>
>
> On 30 January 2017 at 22:17, Shaheed Haque <srhaque at theiet.org> wrote:
>
>> I also put out https://github.com/ShaheedHaque/extra-cmake-modules/pull/3
>> .
>>
>>
>> On 29 January 2017 at 21:09, Shaheed Haque <srhaque at theiet.org> wrote:
>>
>>> OK, let's start small: https://github.com/ShaheedHaqu
>>> e/extra-cmake-modules/pull/2.
>>>
>>> Please review...
>>>
>>> On 29 January 2017 at 17:12, Stephen Kelly <steveire at gmail.com> wrote:
>>>
>>>> Shaheed Haque wrote:
>>>>
>>>> > Rewrite? It think its more a case of forking making things diverge
>>>> (if you
>>>> > look, a lot of the changes are formatting, name changes and comments,
>>>> > though I accept that does make for a lot of noise). I have tried VERY
>>>> hard
>>>> > to keep the good changes form both sides...and yes, there ARE a
>>>> reasonable
>>>> > number of actual changes too.
>>>> >
>>>> > Anyway, I was under the impression a squashed single commit was
>>>> preferred,
>>>> > but I can look to pull things out again. I would prefer to work the
>>>> > reviews via github's Pull Requests, is that still OK?
>>>>
>>>> Yep, sounds good to me!
>>>>
>>>> Thanks,
>>>>
>>>> Steve.
>>>>
>>>>
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-bindings/attachments/20170203/7f88aa4b/attachment.html>


More information about the Kde-bindings mailing list