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. >>>> >>>> >>> >> >