allthingssecurity commented on code in PR #27693:
URL: https://github.com/apache/camel/pull/27693#discussion_r4246365225
##########
components/camel-pgevent/pom.xml:
##########
@@ -54,6 +54,17 @@
<artifactId>camel-test-junit6</artifactId>
<scope>test</scope>
</dependency>
+ <dependency>
+ <groupId>org.mockito</groupId>
+ <artifactId>mockito-junit-jupiter</artifactId>
Review Comment:
Not changed: `mockito-junit-jupiter` is not in the `dependencyManagement` of
`parent/pom.xml` (it only defines the
`mockito-version` property; `assertj-core` is managed there, which is why
that one needs no version). The 53 other
component modules that use `mockito-junit-jupiter` all declare
`<version>${mockito-version}</version>` in the same
way, so the version cannot drift from the rest of the project, and without
it the dependency would have no version.
(Checked by reading the poms on this branch; I did not run a build without
the version.)
_Claude Code on behalf of allthingssecurity_
--
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]
