gemmellr commented on code in PR #4251:
URL: https://github.com/apache/activemq-artemis/pull/4251#discussion_r992109102
##########
pom.xml:
##########
@@ -1029,7 +1028,7 @@
<dependency>
<groupId>com.fasterxml.jackson.core</groupId>
<artifactId>jackson-databind</artifactId>
- <version>${jackson-databind.version}</version>
+ <version>${jackson.version}</version>
Review Comment:
These 2 versions end up differing often enough (and are likely to again
soon..) that I think its worth keeping the separate version property and
defaulting it to the others value when they match. I'd undo this change and
update the property declarations more along the lines of:
<jackson.version>2.13.4</jackson.version>
<jackson-databind.version>${jackson.version}</jackson-databind.version>
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]