Re: Review for PR

[sebb]

On 25 August 2016 at 17:04, Raviteja Lokineni
 wrote:
> The pull request is closed. If you like people to review and merge it:
>
>1. It should be open
>2. There should be an associated JIRA issue

Good advice.

Also note that most developers work on ASF projects in their own time,
so bugs may take a while to get fixed.

As it happens, this one was trivial (no testing needed) so I fixed it
in SVN already, so no need to raise the JIRA.

In general it's hard work keeping track of PRs/fixes sent to the mailing list.
Unless they are dealt with immediately they are likely to get forgotten.


> Thanks,
> Raviteja
>
> On Wed, Aug 24, 2016 at 7:37 AM, Utsav Oza  wrote:
>
>> While surfing through the code base I encountered few spelling errors in
>> the comments provided alongside the code. Please review this pull request
>> and let me know if I am attempting anything in a wrong way or not.
>>
>> https://github.com/apache/commons-net/pull/4
>>
>> Regards
>> Utsav
>>
>
>
>
> --
> *Raviteja Lokineni* | Business Intelligence Developer
> TD Ameritrade
>
> E: raviteja.lokin...@gmail.com
>
> [image: View Raviteja Lokineni's profile on LinkedIn]
> 

-
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Re: Review for PR

[Raviteja Lokineni]

The pull request is closed. If you like people to review and merge it:

   1. It should be open
   2. There should be an associated JIRA issue

Thanks,
Raviteja

On Wed, Aug 24, 2016 at 7:37 AM, Utsav Oza  wrote:

> While surfing through the code base I encountered few spelling errors in
> the comments provided alongside the code. Please review this pull request
> and let me know if I am attempting anything in a wrong way or not.
>
> https://github.com/apache/commons-net/pull/4
>
> Regards
> Utsav
>



-- 
*Raviteja Lokineni* | Business Intelligence Developer
TD Ameritrade

E: raviteja.lokin...@gmail.com

[image: View Raviteja Lokineni's profile on LinkedIn]