<div dir="ltr"><div><div><div>Thanks for your response.<br></div>Now I understand what's going on :)<br><br></div>Cheers,<br></div>Steven Valsesia<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-03-28 13:28 GMT+01:00 Lex Trotman <span dir="ltr"><<a href="mailto:elextr@gmail.com" target="_blank">elextr@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 28 March 2014 22:21, Steven VALSESIA <<a href="mailto:steven.valsesia@gmail.com">steven.valsesia@gmail.com</a>> wrote:<br>

> Hi everybody !<br>
><br>
> I checked all my PR, didn't see any comment, but they're still not merged<br>
> and i don't if there is a reason, or anything else.<br>
> Can someone can check them ? :)<br>
><br>
> I think it would be nice to merge the PR#224, since it already been a<br>
> discussed PR !<br>
<br>
</div>Patience is a virtue, or so they tell me, I wouldn't know :)<br>
<br>
Its only 3 days since you last updated #224.<br>
<br>
There needs to be enough time for people with varying amounts of free<br>
time to comment. (nobody does Geany as a job).<br>
<br>
We need to allow at least a week in most cases.  If its big, difficult<br>
or controversial expect at least a month ... or more ... much more :)<br>
<div class=""><br>
> I know the 2 other PR are maybe not good enough to be merged :)<br>
><br>
> <a href="https://github.com/geany/geany/pull/224" target="_blank">https://github.com/geany/geany/pull/224</a><br>
> <a href="https://github.com/geany/geany/pull/219" target="_blank">https://github.com/geany/geany/pull/219</a> (make the string more accurate with<br>
> PR 218)<br>
> <a href="https://github.com/geany/geany/pull/218" target="_blank">https://github.com/geany/geany/pull/218</a> (let the user choose between Windows<br>
> or Gtk color chooser)<br>
<br>
</div>The last two are both windows.  None of the devs uses windows<br>
regularly (and some like me refuse categorically to do so) so windows<br>
changes will likely be slower than Linux ones anyway, simply because<br>
fewer people will look at them and they need to make a special effort<br>
to do so.<br>
<br>
Also we are just before a release so its a bad idea to commit things<br>
that might introduce errors, since there is not much testing going to<br>
happen before release, and even less on windows changes.<br>
<br>
Thats not a comment on any perceived quality of those PRs.  But<br>
anybody can make a mistake, thats why 1.23.1 followed 1.23 so quickly<br>
last time, after a last minute change caused problems. This time<br>
hopefully no last minute buggy changes will get committed :)<br>
<br>
<br>
Cheers<br>
Lex<br>
<br>
><br>
> Cheers,<br>
> Steven Valsesia<br>
><br>
> _______________________________________________<br>
> Devel mailing list<br>
> <a href="mailto:Devel@lists.geany.org">Devel@lists.geany.org</a><br>
> <a href="https://lists.geany.org/cgi-bin/mailman/listinfo/devel" target="_blank">https://lists.geany.org/cgi-bin/mailman/listinfo/devel</a><br>
><br>
_______________________________________________<br>
Devel mailing list<br>
<a href="mailto:Devel@lists.geany.org">Devel@lists.geany.org</a><br>
<a href="https://lists.geany.org/cgi-bin/mailman/listinfo/devel" target="_blank">https://lists.geany.org/cgi-bin/mailman/listinfo/devel</a><br>
</blockquote></div><br></div>