+1, great idea Not sure it was mentioned before or not but would also be nice to have something like help:reformat goal for juggling things around in the pom to the correct format. Maybe it would be more appropriate to clean up and finish off some of the things I had worked on with the pom plugin in the sandbox and add this as a mojo in there. pom:reformat would be pretty nice
cheers jesse On Wed, Jun 25, 2008 at 10:20 AM, Dennis Lundberg <[EMAIL PROTECTED]> wrote: > Benjamin Bentmann wrote: >> >> Vincent Siveton schrieb: >> >>> I propose to improve [1] to add a code style for our pom files. >> >> +1 for some convention (which is documented as such). > > Yep, +1 for a POM code style > >>> I see the following in this order. >> >> Considering other POMs like parent POMs and Mojo, your proposal should >> finally cover all POM elements (e.g. <packaging>, <url>, <licenses>, >> <developers> etc.). >> >> Whatever we agree upon, we might want to update the Maven Model Reference >> [0], too. I guess that was the ordering Dennis has chosen. > > I've always followed the ordering that is in the Model Reference. > > I don't have a special preference for which order the elements should be in > and I'm happy with what we have now. If we change the order of the elements > in the POM, then there will be a lot of POMs to change. So I say we apply > the path of least resistance and just document that our "style" is that POM > elements should be ordered according to the order they have in the POM > reference and be done with it. > > >> If not already written down, the indentation style (2 spaces) should be >> mentioned, too. > > +1 for that too > > Additional things we could add: > > - Line breaks, when to use them and how > - <project> element always on one line > - white space between sections/elements > >>> <project> >>> ... >>> </project> >> >> I find your proposed ordering more intuitive than the current Model Ref. >> >> >> Benjamin >> >> >> [0] http://maven.apache.org/ref/current/maven-model/maven.html >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [EMAIL PROTECTED] >> For additional commands, e-mail: [EMAIL PROTECTED] >> >> > > > -- > Dennis Lundberg > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- jesse mcconnell [EMAIL PROTECTED] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
