alien11689 commented on code in PR #299: URL: https://github.com/apache/aries/pull/299#discussion_r1903328157
########## blueprint/blueprint-spring/pom.xml: ########## @@ -119,13 +119,13 @@ <dependency> <groupId>org.springframework</groupId> <artifactId>spring-context</artifactId> - <version>4.2.2.RELEASE</version> + <version>6.1.15</version> Review Comment: There are a lot of possible users who stay with Java 8 so it's better to keep all the artifacts compatible with Java 8 if possible. Especially that Spring is not often used in OSGi projects. But personally I see the value in using the newest version of Spring. Maybe the solution here may be creating the new artifact `blueprint-spring6`. Existing `blueprint-spring` could use newest spring 5.x version supporting Java 8. Perfectly the code should be reused between modules so the new module should only override necessary parts (if possible). WDYT? -- 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: dev-unsubscr...@aries.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org