Just to pass on for those who don't read IRC, Colomban expects to add
his input to this thread in a few days after he has finished some
other (real world) commitments.
Cheers
Lex
___
Devel mailing list
Devel@lists.geany.org
https://lists.geany.org/cgi-bi
[...]
>
> I'm afraid I won't remember all those acronyms :-). Maybe just using plain
> speech is fine.
>
Yes, as I said in another post:
> Just say it in plain old English :) "I (do/do not) like the idea, I
> (have/have not) reviewed the implementation and I (have/have not)
> tested it on (win/li
Hi Matthew,
On Wed, Jan 6, 2016 at 4:32 AM, Matthew Brush wrote:
> On 2016-01-05 12:46 PM, Jiří Techet wrote:
>
>> Hi,
>>
>> happy new year and let's celebrate it with something cheerful - zombies!
>>
>>
> Wouldn't they only be zombies if we closed them and they re-opened
> themselves? :)
Ah,
Hi Lex,
On Wed, Jan 6, 2016 at 4:12 AM, Lex Trotman wrote:
> Hi Jiri,
>
> Its a worthwhile thing to talk about. Some specific comments below,
> but first a couple of general ones.
>
> I occasionally talk to some of the Geany devs/contributors in other
> forums and its clear that at this moment G
>
> The advantage of labels is that you see them on the PR overview. Whereas
> with comments you have to open every single PR and look for strings that may
> indicate a review result. While labels can be named arbitrarily, in the end
> there's going to be a handful common ones which can be looked o
On 2016-01-06 03:47 AM, Lex Trotman wrote:
On 6 January 2016 at 20:44, Thomas Martitz wrote:
[...]
I agree that PRs should be merged earlier, with possible fix-up/follow-up
commits in a new PR. This way changes acutally get the testing they need.
The problem with a development branch and with
Am 06.01.2016 um 12:47 schrieb Lex Trotman:
I can't put labels on github. The feature seems to be limited to those with
write access to the repository, so it's useless if we want more reviewers.
Please just add a comment, its actually clearer anyway since we don't
have a well defined semantics
Am 06.01.2016 um 12:39 schrieb Matthew Brush:
On 2016-01-06 02:44 AM, Thomas Martitz wrote:
Am 06.01.2016 um 04:32 schrieb Matthew Brush:
Agree, I sometimes avoid putting LGTM when I think something is a good
idea, because I don't want to give the impression that I have (or even
will) reviewed
On 6 January 2016 at 20:44, Thomas Martitz wrote:
> Am 06.01.2016 um 04:32 schrieb Matthew Brush:
>>
>>
>> Agree, I sometimes avoid putting LGTM when I think something is a good
>> idea, because I don't want to give the impression that I have (or even will)
>> reviewed or tested it. Maybe just a "
On 2016-01-06 02:44 AM, Thomas Martitz wrote:
Am 06.01.2016 um 04:32 schrieb Matthew Brush:
Agree, I sometimes avoid putting LGTM when I think something is a good
idea, because I don't want to give the impression that I have (or even
will) reviewed or tested it. Maybe just a "thumbs up" could m
Am 06.01.2016 um 04:32 schrieb Matthew Brush:
Agree, I sometimes avoid putting LGTM when I think something is a good
idea, because I don't want to give the impression that I have (or even
will) reviewed or tested it. Maybe just a "thumbs up" could mean "good
idea, though I haven't reviewed or
On 2016-01-05 12:46 PM, Jiří Techet wrote:
Hi,
happy new year and let's celebrate it with something cheerful - zombies!
Wouldn't they only be zombies if we closed them and they re-opened
themselves? :)
I've noticed there are more and more pull requests on github which don't
get merged to
Hi Jiri,
Its a worthwhile thing to talk about. Some specific comments below,
but first a couple of general ones.
I occasionally talk to some of the Geany devs/contributors in other
forums and its clear that at this moment Geany isn't their primary
focus. That is my situation, which is why I conc
Hi Jiri,
Happy new year to you too! It's been much work that needed to be finished
before the year's end, and so the pull request has been delayed, that we
were talking about. I'll get to it in a few days though. Considering the
mentioned zombies there seem to be little hurry at the moment.
Cheer
Hi,
happy new year and let's celebrate it with something cheerful - zombies!
I've noticed there are more and more pull requests on github which don't
get merged to Geany. It's clear that people are fighting with time to make
reviews of pull requests (and Colomban does a great job here!), however,
15 matches
Mail list logo