On Thursday 06 April 2017 21:59:34 Julian Sedding wrote: > The configuration property "user.default.mapping" should be called " > use.default.mapping", or maybe rather "default.mapping.enabled" to avoid > typos and confusion.
We already messed up property names in component configurations (incl. typos), but as Julian opened that topic here are my € 0.02: use dots to create a (hierarchical) structure and camel case or snake case to compound separate words – like logging configs in any application, e.g. Tomcat https://tomcat.apache.org/tomcat-8.0-doc/logging.html (-> useDefaultMapping or default_mapping_enabled) Regards, O. > Given this and Olli's comment in JIRA regarding the colon in the principal > name, I think it would be better to cancel the release, fix and reroll. > > Regards > Julian > > On Thursday, April 6, 2017, Daniel Klco <dk...@apache.org> wrote: > > +1 checked signatures and build > > > > On Thu, Apr 6, 2017 at 12:46 AM, Carsten Ziegeler <cziege...@apache.org > > <javascript:;>> > > > > wrote: > > > +1 > > > > > > > > > > > > -- > > > Carsten Ziegeler > > > Adobe Research Switzerland > > > cziege...@apache.org <javascript:;>
