ljacqu

Results 74 issues of ljacqu

Goal: base bean properties on a class's fields (that have getters/setters) rather than using JavaBean definitions. The current concept has been outgrown now that we support `@Comment` on the field—it's...

beanmapper
architecture

It would be useful to run Sonar over this project and get its results. CodeClimate has a plugin at https://github.com/codeclimate/codeclimate-sonar-java, but it seems pretty dead and from what I've understood,...

I might want to have an int property that can only be, for example, between 0 and 10. It would be nice to just let ConfigMe handle it and to...

Not planning to switch soon, but to keep some improvements/adjustments that we could make after switching: - See comments in code referencing this issue - Use `List.of` and `Set.of` where...

I've changed the inline array property type to also consider a final empty entry if the string ends with the separator. This is only (really) relevant for the string type,...

I think it makes sense to have default values of list properties, set properties and similar immutable, but it's a bit cumbersome to have to work around the _current_ value...

architecture

I've come across a lot of configurations that have optional limits that can be disabled by `0` or `-1`. This could be represented as a `Property` so that the application...

Not sure about this one yet, but on PropertyReader: `getKeys` and `getChildKeys` is not used by ConfigMe, and I've been throwing `UnsupportedOperationException` in custom reader implementations I've created. It might...

architecture

Footer comments were added in one of the forks: https://github.com/SkinsRestorer/ConfigMe/commit/82817446380dd68fb71fc9aa90ca7b38c2183839#diff-3ffbfe7dc32739c0f967addeb0cc4cad2edb3bbe196d94d14f43439a08ce6e1f We can easily add this to ConfigMe. ## To do - Extend `CommentsConfiguration` to be able to store footer comments...

good-first-issue

Especially for beans it might be interesting to only have the default value be created if it's really needed, i.e. to supply a `Supplier` for the default value. Downsides: -...

architecture