Harmeet,
> > Removing patch that was not submitted for review,
>
> 1. It was not a patch
>
Yes, it was. It was a modification to the existing code base. Ergo, it
was a patch. That's why we use the [PATCH] flag.
> > introduced inappropriate files into the repository,
>
> 2. Test files are not inappropriate. They should be in the repository,
> maybe
> you mean files not in the right place.
That's what inappropriate means.
> > and violated James stated coding standards.
> 3. What coding standard are you talking about ?
> There is one here: http://jakarta.apache.org/james/code-standards.html
> Can't see anything that collides with this. Do you know of another
coding
> standard ?
>
This is the very coding standard. Please read point #4. Your methods
> It is ok to remove if you have such strong opinions, but not ok to be
so
> far
> from exact.
>
This is ridiculous. We spent three or four emails going over what was
essentially a point or two. I don't know how I could've been any more
exact.
--Peter
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
