Mailing-List: contact dev-help@cxf.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cxf.apache.org
Content-Type: text/plain; charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 8.2 \(2090\))
Subject: Re: what is the current status of checkstyle issue in cxf master?
From: Daniel Kulp <dkulp@apache.org>
In-Reply-To: 
 <CAF8t5XuYB7sW0PuRctz9GXc56fddWm+H66a4ssk5kz2MCL9e8g@mail.gmail.com>
Date: Fri, 20 Mar 2015 19:37:56 -0400
Content-Transfer-Encoding: quoted-printable
Message-Id: <B73D3B7F-15E5-407B-89E9-4F4DBE651FFC@apache.org>
References: 
 <CAF8t5XuYB7sW0PuRctz9GXc56fddWm+H66a4ssk5kz2MCL9e8g@mail.gmail.com>
To: dev@cxf.apache.org,
 Aki Yoshida <elakito@gmail.com>


Check style should be OK.  I updated mine this morning and other than =
flagging a few errors that the previous version did not, it seems OK. =20=


PMD is usually the bigger issue.  I=E2=80=99m using a version of the PMD =
plugin I built from the source because it contains a TON of performance =
fixes that make it much less crappy.   I=E2=80=99m hoping they get a new =
version out soon.   Might need to ping them.

Dan


> On Mar 20, 2015, at 8:15 AM, Aki Yoshida <elakito@gmail.com> wrote:
>=20
> Hi,
> Yesterday, I upgraded the checkstyle plugin to 6.2 by accident in my
> eclipse setup while intending to upgrade other plugins. After that I
> started to get a warning pop up every time I tried to save the working
> file and it was awful. Then, I remembered this Dan's mail from a while
> ago.
> =
http://cxf.547215.n5.nabble.com/Warning-Don-t-update-your-eclipse-Checksty=
le-plugin-td5754012.html
>=20
> So finally, I uninstalled the plugin.
> I don't know if I had to do some clean up something or other
> refreshing to avoid this problem to use the current checkstyle?
>=20
> regards, aki

--=20
Daniel Kulp
dkulp@apache.org - http://dankulp.com/blog
Talend Community Coder - http://coders.talend.com