Re: PyKF5 bindings generator improvements out for review.

Shaheed Haque <[email protected]>
Newsgroups gmane.comp.kde.devel.bindings
Message-ID <CAHAc2jcAyTUDX8ZiaYLxiJiM+X2Dp-O1ty0MUJwfsN-CySMJOA@mail.gmail.com>
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 <[email protected]> wrote:

> And now https://github.com/ShaheedHaque/extra-cmake-modules/pull/4...
>
>
> On 30 January 2017 at 22:17, Shaheed Haque <[email protected]> wrote:
>
>> I also put out https://github.com/ShaheedHaque/extra-cmake-modules/pull/3
>> .
>>
>>
>> On 29 January 2017 at 21:09, Shaheed Haque <[email protected]> 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 <[email protected]> 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.
>>>>
>>>>
>>>
>>
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.