Re: [mojo-dev] Re: [mojo-scm] [19715] trunk/mojo/flatten-maven-plugin/src/site/xdoc/usage.xml: MOJO-2027: added documentation

[Jörg Hohwiller]

Am 24.05.2014 01:26, schrieb Sergei Ivanov:

Hello Jörg,

Hi Sergei,

I have two minor suggestions:
1. Rename flattenDescriptor to pomSections (or pomElements)
2. Start enumerated constants with a lower case letter, e.g. keepIfExists, so 
that they are consistent with other mojos' configuration

The reason for the above is that I think it reads better that way:

<configuration>
    <pomSections>
      <name>keepOrAdd</name>
      <description>keepIfExists</description>
      <url>remove</url>
      ...
    </pomSections>
</configuration>

Thanks for you suggestion. Makes sense and I will go for both.
I would prefere pomElements.

Kind regards,
--
Sergei Ivanov

Cheers
  Jörg

smime.p7s
Description: S/MIME Cryptographic Signature