Some trivial tickets

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
9 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Some trivial tickets

Yazad Khambata
Hi,

I have been looking around to see if there some simple items I can
pick up to make some initial contributions. I see 3 trivial tickets
that I can work on if that's ok,

GROOVY-7388
DefaultGroovyMethods class's plus(T[] left, xxxx<T> right) have ambiguous errors

In this ticket I see Paul has requested a test case and has not
received a response, I could look at providing a test case and a fix
for this. Note I see that there also a duplicate filed but that is
closed without any work-log or comments.

GROOVY-5227
AbstractHttpServlet should extend GenericServlet directly instead of HttpServlet

Looks like this is one of the tickets where I may be able to provide
some analysis and suggestions and based on that a decision can be made
to either close it or implement it.

Both tickets have had some comments in 2016 but don't seem to have an
owner. Please let me know if I can pick them up. Thank you.

Regards,
Yazad Khambata
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Some trivial tickets

Yazad Khambata
Hi Paul,

Did you get a chance to look at this mail? I noticed you have sent out
a mail to mark some tickets as low hanging fruits, did you want me to
have a look at that instead? Thank you.

Regards,


On Tue, May 2, 2017 at 8:40 AM, Yazad Khambata <[hidden email]> wrote:

> Hi,
>
> I have been looking around to see if there some simple items I can
> pick up to make some initial contributions. I see 3 trivial tickets
> that I can work on if that's ok,
>
> GROOVY-7388
> DefaultGroovyMethods class's plus(T[] left, xxxx<T> right) have ambiguous errors
>
> In this ticket I see Paul has requested a test case and has not
> received a response, I could look at providing a test case and a fix
> for this. Note I see that there also a duplicate filed but that is
> closed without any work-log or comments.
>
> GROOVY-5227
> AbstractHttpServlet should extend GenericServlet directly instead of HttpServlet
>
> Looks like this is one of the tickets where I may be able to provide
> some analysis and suggestions and based on that a decision can be made
> to either close it or implement it.
>
> Both tickets have had some comments in 2016 but don't seem to have an
> owner. Please let me know if I can pick them up. Thank you.
>
> Regards,
> Yazad Khambata
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Some trivial tickets

Yazad Khambata
In reply to this post by Yazad Khambata
+group

On Thu, May 4, 2017 at 6:00 AM Yazad Khambata <[hidden email]> wrote:
Hi Paul,

Awesome, I will provide my analysis on GROOVY-5227 soon. Thank you!

Regards,

On Thu, May 4, 2017 at 2:06 AM Paul King <[hidden email]> wrote:

Hi Yazad,

I wouldn't bother with GROOVY-7388. I can imagine some scenarios where ambiguous messages might occur but they wouldn't involve normal usage of those methods. I have closed for now. If the OP provides more information we can pursue that one.

GROOVY-5227 seems worthy of investigation. An analysis would be great.

Hopefully some more tickets will be ready over the next few days based on my other email.

Cheers, Paul.


On Wed, May 3, 2017 at 1:40 AM, Yazad Khambata <[hidden email]> wrote:
Hi,

I have been looking around to see if there some simple items I can
pick up to make some initial contributions. I see 3 trivial tickets
that I can work on if that's ok,

GROOVY-7388
DefaultGroovyMethods class's plus(T[] left, xxxx<T> right) have ambiguous errors

In this ticket I see Paul has requested a test case and has not
received a response, I could look at providing a test case and a fix
for this. Note I see that there also a duplicate filed but that is
closed without any work-log or comments.

GROOVY-5227
AbstractHttpServlet should extend GenericServlet directly instead of HttpServlet

Looks like this is one of the tickets where I may be able to provide
some analysis and suggestions and based on that a decision can be made
to either close it or implement it.

Both tickets have had some comments in 2016 but don't seem to have an
owner. Please let me know if I can pick them up. Thank you.

Regards,
Yazad Khambata

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Some trivial tickets

bo zhang
I have no idea about that issue, but I'm impressed by your enthusiasm thus I want to encourage you to advance bravely. There's nobody responding to you, maybe these bugs are too old to be recalled by people? I hope you're not discouraged.

Just like you, I tried to make my first contribution several months ago, and I picked up some Critical and Major bugs to fix, which were actually pretty to easy. So what I want to say is don't be scared by the Priority, maybe you can try some other bugs you're interested in?

I believe you can make perfect contributions. Good luck to you.


On Thu, May 4, 2017 at 9:01 PM, Yazad Khambata <[hidden email]> wrote:
+group

On Thu, May 4, 2017 at 6:00 AM Yazad Khambata <[hidden email]> wrote:
Hi Paul,

Awesome, I will provide my analysis on GROOVY-5227 soon. Thank you!

Regards,

On Thu, May 4, 2017 at 2:06 AM Paul King <[hidden email]> wrote:

Hi Yazad,

I wouldn't bother with GROOVY-7388. I can imagine some scenarios where ambiguous messages might occur but they wouldn't involve normal usage of those methods. I have closed for now. If the OP provides more information we can pursue that one.

GROOVY-5227 seems worthy of investigation. An analysis would be great.

Hopefully some more tickets will be ready over the next few days based on my other email.

Cheers, Paul.


On Wed, May 3, 2017 at 1:40 AM, Yazad Khambata <[hidden email]> wrote:
Hi,

I have been looking around to see if there some simple items I can
pick up to make some initial contributions. I see 3 trivial tickets
that I can work on if that's ok,

GROOVY-7388
DefaultGroovyMethods class's plus(T[] left, xxxx<T> right) have ambiguous errors

In this ticket I see Paul has requested a test case and has not
received a response, I could look at providing a test case and a fix
for this. Note I see that there also a duplicate filed but that is
closed without any work-log or comments.

GROOVY-5227
AbstractHttpServlet should extend GenericServlet directly instead of HttpServlet

Looks like this is one of the tickets where I may be able to provide
some analysis and suggestions and based on that a decision can be made
to either close it or implement it.

Both tickets have had some comments in 2016 but don't seem to have an
owner. Please let me know if I can pick them up. Thank you.

Regards,
Yazad Khambata


Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Some trivial tickets

Yazad Khambata
Thanks a lot Bo, actually Paul responded to me this morning and I noticed just forwarded to the whole team for the record. I think I broke the chain while forwarding. Sorry!

Thank you for the kind words so far I have interacted with Paul and you and I feel very welcomed. I am sure I have a lot to learn from the teams collective experience. Thank you!

Regards,
Yazad Khambata

On Thu, May 4, 2017 at 8:51 AM bo zhang <[hidden email]> wrote:
I have no idea about that issue, but I'm impressed by your enthusiasm thus I want to encourage you to advance bravely. There's nobody responding to you, maybe these bugs are too old to be recalled by people? I hope you're not discouraged.

Just like you, I tried to make my first contribution several months ago, and I picked up some Critical and Major bugs to fix, which were actually pretty to easy. So what I want to say is don't be scared by the Priority, maybe you can try some other bugs you're interested in?

I believe you can make perfect contributions. Good luck to you.


On Thu, May 4, 2017 at 9:01 PM, Yazad Khambata <[hidden email]> wrote:
+group

On Thu, May 4, 2017 at 6:00 AM Yazad Khambata <[hidden email]> wrote:
Hi Paul,

Awesome, I will provide my analysis on GROOVY-5227 soon. Thank you!

Regards,

On Thu, May 4, 2017 at 2:06 AM Paul King <[hidden email]> wrote:

Hi Yazad,

I wouldn't bother with GROOVY-7388. I can imagine some scenarios where ambiguous messages might occur but they wouldn't involve normal usage of those methods. I have closed for now. If the OP provides more information we can pursue that one.

GROOVY-5227 seems worthy of investigation. An analysis would be great.

Hopefully some more tickets will be ready over the next few days based on my other email.

Cheers, Paul.


On Wed, May 3, 2017 at 1:40 AM, Yazad Khambata <[hidden email]> wrote:
Hi,

I have been looking around to see if there some simple items I can
pick up to make some initial contributions. I see 3 trivial tickets
that I can work on if that's ok,

GROOVY-7388
DefaultGroovyMethods class's plus(T[] left, xxxx<T> right) have ambiguous errors

In this ticket I see Paul has requested a test case and has not
received a response, I could look at providing a test case and a fix
for this. Note I see that there also a duplicate filed but that is
closed without any work-log or comments.

GROOVY-5227
AbstractHttpServlet should extend GenericServlet directly instead of HttpServlet

Looks like this is one of the tickets where I may be able to provide
some analysis and suggestions and based on that a decision can be made
to either close it or implement it.

Both tickets have had some comments in 2016 but don't seem to have an
owner. Please let me know if I can pick them up. Thank you.

Regards,
Yazad Khambata


Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Fwd: Some trivial tickets

Yazad Khambata
In reply to this post by Yazad Khambata
Forwarded broken chain earlier.

---------- Forwarded message ---------
From: Yazad Khambata <[hidden email]>
Date: Thu, May 4, 2017 at 6:00 AM
Subject: Re: Some trivial tickets
To: <[hidden email]>


Hi Paul,

Awesome, I will provide my analysis on GROOVY-5227 soon. Thank you!

Regards,

On Thu, May 4, 2017 at 2:06 AM Paul King <[hidden email]> wrote:

Hi Yazad,

I wouldn't bother with GROOVY-7388. I can imagine some scenarios where ambiguous messages might occur but they wouldn't involve normal usage of those methods. I have closed for now. If the OP provides more information we can pursue that one.

GROOVY-5227 seems worthy of investigation. An analysis would be great.

Hopefully some more tickets will be ready over the next few days based on my other email.

Cheers, Paul.


On Wed, May 3, 2017 at 1:40 AM, Yazad Khambata <[hidden email]> wrote:
Hi,

I have been looking around to see if there some simple items I can
pick up to make some initial contributions. I see 3 trivial tickets
that I can work on if that's ok,

GROOVY-7388
DefaultGroovyMethods class's plus(T[] left, xxxx<T> right) have ambiguous errors

In this ticket I see Paul has requested a test case and has not
received a response, I could look at providing a test case and a fix
for this. Note I see that there also a duplicate filed but that is
closed without any work-log or comments.

GROOVY-5227
AbstractHttpServlet should extend GenericServlet directly instead of HttpServlet

Looks like this is one of the tickets where I may be able to provide
some analysis and suggestions and based on that a decision can be made
to either close it or implement it.

Both tickets have had some comments in 2016 but don't seem to have an
owner. Please let me know if I can pick them up. Thank you.

Regards,
Yazad Khambata

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Some trivial tickets

Yazad Khambata
Hi Paul,

Just wanted to check if you are good with the analysis provided for,

GROOVY-5227 - AbstractHttpServlet should extend GenericServlet
directly instead of HttpServlet

Please suggest if you'd like to fix or close based on the analysis and
suggestions provided in the comments.

Also as the next ticket I was wondering if I could take up,

GROOVY-6485 - Replace usage of String#toUpperCase() with
String#toUpperCase(Locale.ENGLISH)

Thank you.

Regards,
Yazad Khambata



On Thu, May 4, 2017 at 8:56 AM, Yazad Khambata <[hidden email]> wrote:

> Forwarded broken chain earlier.
>
> ---------- Forwarded message ---------
> From: Yazad Khambata <[hidden email]>
> Date: Thu, May 4, 2017 at 6:00 AM
> Subject: Re: Some trivial tickets
> To: <[hidden email]>
>
>
> Hi Paul,
>
> Awesome, I will provide my analysis on GROOVY-5227 soon. Thank you!
>
> Regards,
>
> On Thu, May 4, 2017 at 2:06 AM Paul King <[hidden email]> wrote:
>>
>>
>> Hi Yazad,
>>
>> I wouldn't bother with GROOVY-7388. I can imagine some scenarios where
>> ambiguous messages might occur but they wouldn't involve normal usage of
>> those methods. I have closed for now. If the OP provides more information we
>> can pursue that one.
>>
>> GROOVY-5227 seems worthy of investigation. An analysis would be great.
>>
>> Hopefully some more tickets will be ready over the next few days based on
>> my other email.
>>
>> Cheers, Paul.
>>
>>
>> On Wed, May 3, 2017 at 1:40 AM, Yazad Khambata <[hidden email]> wrote:
>>>
>>> Hi,
>>>
>>> I have been looking around to see if there some simple items I can
>>> pick up to make some initial contributions. I see 3 trivial tickets
>>> that I can work on if that's ok,
>>>
>>> GROOVY-7388
>>> DefaultGroovyMethods class's plus(T[] left, xxxx<T> right) have ambiguous
>>> errors
>>>
>>> In this ticket I see Paul has requested a test case and has not
>>> received a response, I could look at providing a test case and a fix
>>> for this. Note I see that there also a duplicate filed but that is
>>> closed without any work-log or comments.
>>>
>>> GROOVY-5227
>>> AbstractHttpServlet should extend GenericServlet directly instead of
>>> HttpServlet
>>>
>>> Looks like this is one of the tickets where I may be able to provide
>>> some analysis and suggestions and based on that a decision can be made
>>> to either close it or implement it.
>>>
>>> Both tickets have had some comments in 2016 but don't seem to have an
>>> owner. Please let me know if I can pick them up. Thank you.
>>>
>>> Regards,
>>> Yazad Khambata
>>
>>
>
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Some trivial tickets

paulk_asert
I'll make some time tomorrow to take a look.

Cheers, Paul.

On Mon, Jun 19, 2017 at 12:37 PM, Yazad Khambata <[hidden email]> wrote:
Hi Paul,

Just wanted to check if you are good with the analysis provided for,

GROOVY-5227 - AbstractHttpServlet should extend GenericServlet
directly instead of HttpServlet

Please suggest if you'd like to fix or close based on the analysis and
suggestions provided in the comments.

Also as the next ticket I was wondering if I could take up,

GROOVY-6485 - Replace usage of String#toUpperCase() with
String#toUpperCase(Locale.ENGLISH)

Thank you.

Regards,
Yazad Khambata



On Thu, May 4, 2017 at 8:56 AM, Yazad Khambata <[hidden email]> wrote:
> Forwarded broken chain earlier.
>
> ---------- Forwarded message ---------
> From: Yazad Khambata <[hidden email]>
> Date: Thu, May 4, 2017 at 6:00 AM
> Subject: Re: Some trivial tickets
> To: <[hidden email]>
>
>
> Hi Paul,
>
> Awesome, I will provide my analysis on GROOVY-5227 soon. Thank you!
>
> Regards,
>
> On Thu, May 4, 2017 at 2:06 AM Paul King <[hidden email]> wrote:
>>
>>
>> Hi Yazad,
>>
>> I wouldn't bother with GROOVY-7388. I can imagine some scenarios where
>> ambiguous messages might occur but they wouldn't involve normal usage of
>> those methods. I have closed for now. If the OP provides more information we
>> can pursue that one.
>>
>> GROOVY-5227 seems worthy of investigation. An analysis would be great.
>>
>> Hopefully some more tickets will be ready over the next few days based on
>> my other email.
>>
>> Cheers, Paul.
>>
>>
>> On Wed, May 3, 2017 at 1:40 AM, Yazad Khambata <[hidden email]> wrote:
>>>
>>> Hi,
>>>
>>> I have been looking around to see if there some simple items I can
>>> pick up to make some initial contributions. I see 3 trivial tickets
>>> that I can work on if that's ok,
>>>
>>> GROOVY-7388
>>> DefaultGroovyMethods class's plus(T[] left, xxxx<T> right) have ambiguous
>>> errors
>>>
>>> In this ticket I see Paul has requested a test case and has not
>>> received a response, I could look at providing a test case and a fix
>>> for this. Note I see that there also a duplicate filed but that is
>>> closed without any work-log or comments.
>>>
>>> GROOVY-5227
>>> AbstractHttpServlet should extend GenericServlet directly instead of
>>> HttpServlet
>>>
>>> Looks like this is one of the tickets where I may be able to provide
>>> some analysis and suggestions and based on that a decision can be made
>>> to either close it or implement it.
>>>
>>> Both tickets have had some comments in 2016 but don't seem to have an
>>> owner. Please let me know if I can pick them up. Thank you.
>>>
>>> Regards,
>>> Yazad Khambata
>>
>>
>

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Some trivial tickets

Yazad Khambata
Thank you.

Regards,
Yazad Khambata


On Mon, Jun 19, 2017 at 5:23 AM Paul King <[hidden email]> wrote:
I'll make some time tomorrow to take a look.

Cheers, Paul.

On Mon, Jun 19, 2017 at 12:37 PM, Yazad Khambata <[hidden email]> wrote:
Hi Paul,

Just wanted to check if you are good with the analysis provided for,

GROOVY-5227 - AbstractHttpServlet should extend GenericServlet
directly instead of HttpServlet

Please suggest if you'd like to fix or close based on the analysis and
suggestions provided in the comments.

Also as the next ticket I was wondering if I could take up,

GROOVY-6485 - Replace usage of String#toUpperCase() with
String#toUpperCase(Locale.ENGLISH)

Thank you.

Regards,
Yazad Khambata



On Thu, May 4, 2017 at 8:56 AM, Yazad Khambata <[hidden email]> wrote:
> Forwarded broken chain earlier.
>
> ---------- Forwarded message ---------
> From: Yazad Khambata <[hidden email]>
> Date: Thu, May 4, 2017 at 6:00 AM
> Subject: Re: Some trivial tickets
> To: <[hidden email]>
>
>
> Hi Paul,
>
> Awesome, I will provide my analysis on GROOVY-5227 soon. Thank you!
>
> Regards,
>
> On Thu, May 4, 2017 at 2:06 AM Paul King <[hidden email]> wrote:
>>
>>
>> Hi Yazad,
>>
>> I wouldn't bother with GROOVY-7388. I can imagine some scenarios where
>> ambiguous messages might occur but they wouldn't involve normal usage of
>> those methods. I have closed for now. If the OP provides more information we
>> can pursue that one.
>>
>> GROOVY-5227 seems worthy of investigation. An analysis would be great.
>>
>> Hopefully some more tickets will be ready over the next few days based on
>> my other email.
>>
>> Cheers, Paul.
>>
>>
>> On Wed, May 3, 2017 at 1:40 AM, Yazad Khambata <[hidden email]> wrote:
>>>
>>> Hi,
>>>
>>> I have been looking around to see if there some simple items I can
>>> pick up to make some initial contributions. I see 3 trivial tickets
>>> that I can work on if that's ok,
>>>
>>> GROOVY-7388
>>> DefaultGroovyMethods class's plus(T[] left, xxxx<T> right) have ambiguous
>>> errors
>>>
>>> In this ticket I see Paul has requested a test case and has not
>>> received a response, I could look at providing a test case and a fix
>>> for this. Note I see that there also a duplicate filed but that is
>>> closed without any work-log or comments.
>>>
>>> GROOVY-5227
>>> AbstractHttpServlet should extend GenericServlet directly instead of
>>> HttpServlet
>>>
>>> Looks like this is one of the tickets where I may be able to provide
>>> some analysis and suggestions and based on that a decision can be made
>>> to either close it or implement it.
>>>
>>> Both tickets have had some comments in 2016 but don't seem to have an
>>> owner. Please let me know if I can pick them up. Thank you.
>>>
>>> Regards,
>>> Yazad Khambata
>>
>>
>

Loading...