On 08/15/2016 05:40 PM, Gerald Pfeifer wrote:
> Hi Martin,
>
> On Mon, 15 Aug 2016, Martin Liška wrote:
>> Ready to be installed?
>
> you ignored (or I guess: missed) the updated patch that I
> included in my previous message. Can you use that instead
> of your original?
Ah, sorry for not adopt
Hi Martin,
On Mon, 15 Aug 2016, Martin Liška wrote:
> Ready to be installed?
you ignored (or I guess: missed) the updated patch that I
included in my previous message. Can you use that instead
of your original?
As for the examples, I would omit the output of the script,
since otherwise we'll ne
On 08/14/2016 06:35 PM, Gerald Pfeifer wrote:
> The one thing that would be really good to add is documentation on
> the usage of the script and/or examples for relevant invocations.
>
> Thanks,
> Gerald
Good idea, sending second version of the patch where I added usage samples.
Ready to be inst
Hi Martin,
On Fri, 12 Aug 2016, Martin Liška wrote:
>> I think it would be good to add a comment at the top that
>> describes what the scripts does and how to invoke the script?
> Sure, what about this?
your patch definitely is a nice improvement. Instead of providing
editorial comments, allow m
On 08/09/2016 09:28 AM, Gerald Pfeifer wrote:
> On Wed, 3 Aug 2016, Martin Liška wrote:
>> I've installed (r239066) the script which is used by maintainers to
>> update PRs in a batch mode.
>
> I think it would be good to add a comment at the top that
> describes what the scripts does and how to
On Wed, 3 Aug 2016, Martin Liška wrote:
> I've installed (r239066) the script which is used by maintainers to
> update PRs in a batch mode.
I think it would be good to add a comment at the top that
describes what the scripts does and how to invoke the script?
Gerald
