If the goal is to be able to run a tool to get checkstyle 'conformance'
wouldn't it be better to tailer checkstyle to a tool anyone could grab,
rather than tailoring it to what IDEA can do? Don't get me wrong, I love
IDEA and I'd be quite happy to be able to use it for this, but not
everyone has IDEA...
+1 in principle, though.
Ted Husted wrote:
It's just that I don't have a lot of time right now to try and get
IDEA or Jalopy to match our CheckStyle setup. If CheckStyle is going
to be held up as the arbitrator of coding style, it would be nice if a
code reformatter could reach the CheckStyle configuration directly.
If anyone would like to volunteer to try and match IDEA's
configuration with our CheckStyle file, that would be very helpful. I
made some changes against the IDEA defaults, but apparently they are
not quite right. I'd like to reduce the coding inconsistencies, but I
don't want to introduce a lot of manual changes right before a
release.
The changes from the IDEA norm I've identified so far are documented
in my comment of 12/31.
http://issues.apache.org/bugzilla/show_bug.cgi?id=36306
Or, alternatively, we could develop a CheckStyle file to match the
IDEA norm, and use that instead. I think the important thing is to
have a coding style that we can enforce with an automatic tool, so as
to not waste volunteer time on mechanical tasks.
-Ted.
On 1/17/06, Frank W. Zammetti <[EMAIL PROTECTED]> wrote:
You may want to have a look at Jalopy:
jalopy.sf.net
There are plugins for various IDEs and editors, as well as an Ant task
(IIRC, you can use Ant tasks from Maven?). It won't fix everything, but
it will help enforce many of the more common coding standards. It
should be possible to configure its rules to match those of CheckStyle,
so you could then use CheckStyle to manually deal with what remains that
Jalopy doesn't fix.
Frank
Don Brown wrote:
It just fills the errors list with checkstyle errors, that you can then
click on to be taken to the file and line. It was very handy for
cleaning up Scripting.
Don
Ted Husted wrote:
Does it actually reformat the code? Or just report on what should be
reformatted?
On 1/17/06, Don Brown <[EMAIL PROTECTED]> wrote:
I've been using jEdit's checkstyle plugin. I had to temporary modify
the checkstyle xml to remove those dynamic properties, but otherwise, it
worked great.
Don
Ted Husted wrote:
Is anyone using an automatic code reformatting tool that can read a
CheckStyle configuration?
I'd like to clean up some of the stylistic errors, but I would prefer
that most of the reformatting be done by a proven tool. I have utter
confidence in IDEA, but I don't know if we can perfectly align the
settings with CheckStyle.
For more, see
* http://issues.apache.org/bugzilla/show_bug.cgi?id=36306
Having CheckStyle standards is great, but I would rather have a
consistent code style that we can enforce with an automatic
reformatting tool.
-Ted.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
