[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bindings
Subject:    Re: PyKF5 bindings generator improvements out for review.
From:       Shaheed Haque <srhaque () theiet ! org>
Date:       2017-01-31 22:26:00
Message-ID: CAHAc2jcv8mrqGgbyzfjjCOC-MfaTfOB1unszTSu7R5OoGSRf7Q () mail ! gmail ! com
[Download RAW message or body]

And now https://github.com/ShaheedHaque/extra-cmake-modules/pull/4...

On 30 January 2017 at 22:17, Shaheed Haque <srhaque@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@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@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.
>>>
>>>
>>
>

[Attachment #3 (text/html)]

<div dir="ltr">And now <a \
href="https://github.com/ShaheedHaque/extra-cmake-modules/pull/4.">https://github.com/ShaheedHaque/extra-cmake-modules/pull/4.</a>..<br><div \
class="gmail_extra"><br><div class="gmail_quote">On 30 January 2017 at 22:17, Shaheed \
Haque <span dir="ltr">&lt;<a href="mailto:srhaque@theiet.org" \
target="_blank">srhaque@theiet.org</a>&gt;</span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><div dir="ltr">I also put out <a \
href="https://github.com/ShaheedHaque/extra-cmake-modules/pull/3" \
target="_blank">https://github.com/<wbr>ShaheedHaque/extra-cmake-<wbr>modules/pull/3</a>.<br><br></div><div \
class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div \
class="gmail_quote">On 29 January 2017 at 21:09, Shaheed Haque <span dir="ltr">&lt;<a \
href="mailto:srhaque@theiet.org" target="_blank">srhaque@theiet.org</a>&gt;</span> \
wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px \
#ccc solid;padding-left:1ex"><div dir="ltr"><div>OK, let&#39;s start small: <a \
href="https://github.com/ShaheedHaque/extra-cmake-modules/pull/2" \
target="_blank">https://github.com/ShaheedHaqu<wbr>e/extra-cmake-modules/pull/2</a>.<br><br></div>Please \
review...<br></div><div class="m_2249465440063826934HOEnZb"><div \
class="m_2249465440063826934h5"><div class="gmail_extra"><br><div \
class="gmail_quote">On 29 January 2017 at 17:12, Stephen Kelly <span dir="ltr">&lt;<a \
href="mailto:steveire@gmail.com" target="_blank">steveire@gmail.com</a>&gt;</span> \
wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px \
#ccc solid;padding-left:1ex"><span>Shaheed Haque wrote:<br> <br>
&gt; Rewrite? It think its more a case of forking making things diverge (if you<br>
&gt; look, a lot of the changes are formatting, name changes and comments,<br>
&gt; though I accept that does make for a lot of noise). I have tried VERY hard<br>
&gt; to keep the good changes form both sides...and yes, there ARE a reasonable<br>
&gt; number of actual changes too.<br>
&gt;<br>
&gt; Anyway, I was under the impression a squashed single commit was preferred,<br>
&gt; but I can look to pull things out again. I would prefer to work the<br>
&gt; reviews via github&#39;s Pull Requests, is that still OK?<br>
<br>
</span>Yep, sounds good to me!<br>
<br>
Thanks,<br>
<br>
Steve.<br>
<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div></div>



[prev in list] [next in list] [prev in thread] [next in thread]