[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
[geclipse-dev] Checkstyle config on packages
|
Hi Ariel, Markus, etc,
As I added Checkstyle configuration that was missing on a couple of GRIA
packages in g-Eclipse, I noticed that there were quite a number of other
packages that were not using Checkstyle.
So I've been through and set this up, for at least the core g-Eclipse
code itself (eu.geclipse.*).
There are some others, e.g. org.apache.*, which don't have settings, but
I wasn't sure what your policy was on those packages.
I've checked in all the changes for packages that do not cause any
errors (most of them, fortunately). However, the following packages
cause many errors for me:
eu.geclipse.core.sla
eu.geclipse.glite.ui
eu.geclipse.nextgrid.sla
eu.geclipse.nextgrid.sla.ui
I didn't want to break any current builds, so I haven't checked these
in.
Should I:
A) Check in anyway (causing new errors in current build)
B) Check in after the M8 release
C) Leave them alone
Cheers,
Ken.