Croway commented on PR #134:
URL: 
https://github.com/apache/camel-upgrade-recipes/pull/134#issuecomment-5810635088

   The property-based additions cover the intended Maven property names, but 
the new tests currently expect literal `@camel-spring-boot-version@` and 
`@spring-boot-version@` strings (for example, in 
`propertyManagedBomIsUpdated`). Maven filters the recipe YAML resources, not 
Java test source. The recipe will use concrete target versions while these 
expected POMs still contain `@...@`, so the assertions will not match. Please 
use concrete expected versions or read the target values from the build 
configuration.
   
   _AI-generated review comment on behalf of Federico Mariani._
   


-- 
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]

Reply via email to