Pankraz76 commented on code in PR #2434: URL: https://github.com/apache/maven/pull/2434#discussion_r2128416967
########## src/mdo/java/WrapperProperties.java: ########## @@ -48,9 +48,9 @@ class WrapperProperties extends Properties { - final Supplier<Map<String, String>> getter; - final Consumer<Properties> setter; private final OrderedProperties orderedProps = new OrderedProperties(); Review Comment: this is the main character, other just NPC, only one having init on declaration. might give this special attention having first. ########## src/mdo/java/WrapperProperties.java: ########## @@ -48,9 +48,9 @@ class WrapperProperties extends Properties { - final Supplier<Map<String, String>> getter; - final Consumer<Properties> setter; private final OrderedProperties orderedProps = new OrderedProperties(); Review Comment: this is the main character, others just NPC, only one having init on declaration. might give this special attention having first. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org