+1 Dan submitted a patch with testcases so I think it should go into
the build process with that patch applied. Thanks Dan!
Jim
On Apr 26, 2006, at 11:42 AM, Jean-Sebastien Delfino wrote:
Jim Marino wrote:
Just a friendly reminder to those working on the core or model to
please include celtix in their refactorings. I accidentally forgot
to check in my refactors (sandbox is separate) and run the
checkstyle tests from celtix/binding.celtix resulting in Dan
having a broken demo :-( Also, someone else modifying the loader
forgot to include the project in their refactors (I fixed it)...
Dan also proposed we adopt checkstyle as part of the build
process. I would like to due this using lazy consensus with the
caveat that the cut-over be done after the Java One release to
avoid disruption and give people a chance to adjust code
gradually. The checkstyle configuration is fairly rigorous and we
will need to divide up the work to do so by project.
Jim
Sorry for contributing to the break as well... when I did my latest
refactoring I didn't have the Celtix binding included in my
workspace either. What do people think about moving the Celtix
binding to the main code stream under bindings/? It's getting in a
good shape now, and this way we'll all include it in our
refactoring, testing, it'll be loaded in our IDEs etc.
Any thoughts?
--
Jean-Sebastien
