Re: [PR] [DO-NOT-MERGE][incubator-kie-issues-2204] upgrade Quarkus to 3.27.2 and Spring Boot to 3.5.10 [incubator-kie-optaplanner]
jeejz commented on code in PR #3201: URL: https://github.com/apache/incubator-kie-optaplanner/pull/3201#discussion_r2801707385 ## build/optaplanner-build-parent/pom.xml: ## @@ -46,20 +46,20 @@ 2.22.1 1.4.21 6.6.7 -3.20.3 -4.1.127.Final +3.27.2 +4.1.130.Final Review Comment: This is removed. ## build/optaplanner-build-parent/pom.xml: ## @@ -69,7 +69,7 @@ 2.2.3 3.4.2 -3.8.1 +3.13.0 Review Comment: This is removed. -- 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] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [DO-NOT-MERGE][incubator-kie-issues-2204] upgrade Quarkus to 3.27.2 and Spring Boot to 3.5.10 [incubator-kie-optaplanner]
yesamer commented on code in PR #3201: URL: https://github.com/apache/incubator-kie-optaplanner/pull/3201#discussion_r2798457774 ## build/optaplanner-build-parent/pom.xml: ## @@ -69,7 +69,7 @@ 2.2.3 3.4.2 -3.8.1 +3.13.0 Review Comment: @jeejz Same ## build/optaplanner-build-parent/pom.xml: ## @@ -46,20 +46,20 @@ 2.22.1 1.4.21 6.6.7 -3.20.3 -4.1.127.Final +3.27.2 +4.1.130.Final Review Comment: @jeejz This is a good point, I guess we can remove it. -- 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] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [DO-NOT-MERGE][incubator-kie-issues-2204] upgrade Quarkus to 3.27.2 and Spring Boot to 3.5.10 [incubator-kie-optaplanner]
Copilot commented on code in PR #3201:
URL:
https://github.com/apache/incubator-kie-optaplanner/pull/3201#discussion_r2798296971
##
build/optaplanner-build-parent/pom.xml:
##
@@ -46,20 +46,20 @@
2.22.1
1.4.21
6.6.7
-3.20.3
-4.1.127.Final
+3.27.2
+4.1.130.Final
Review Comment:
`version.io.netty` is defined/updated here but it is not referenced anywhere
else in the repository (no `${version.io.netty}` usages). As a result, this
Netty version bump has no effect on the actual resolved Netty artifacts. Either
remove this unused property, or wire it into dependency/plugin management where
Netty is overridden so the upgrade is effective.
```suggestion
```
##
build/optaplanner-build-parent/pom.xml:
##
@@ -69,7 +69,7 @@
2.2.3
3.4.2
-3.8.1
+3.13.0
Review Comment:
`version.compiler.plugin` is updated but appears to be unused (there is no
`${version.compiler.plugin}` reference and the `maven-compiler-plugin`
declaration in this POM has no ``). If the intent is to manage the
compiler plugin version, reference this property from the
`maven-compiler-plugin` entry in pluginManagement; otherwise drop the property
to avoid confusion.
```suggestion
```
--
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]
-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
Re: [PR] [DO-NOT-MERGE][incubator-kie-issues-2204] upgrade Quarkus to 3.27.2 and Spring Boot to 3.5.10 [incubator-kie-optaplanner]
jeejz commented on code in PR #3201: URL: https://github.com/apache/incubator-kie-optaplanner/pull/3201#discussion_r2773057446 ## build/optaplanner-build-parent/pom.xml: ## @@ -46,20 +46,20 @@ 2.22.1 1.4.21 6.6.7 -3.20.3 -4.1.127.Final +3.27.2 +4.1.130.Final 3.6.1 1.11.0 Review Comment: This is synced to drools now. -- 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] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [DO-NOT-MERGE][incubator-kie-issues-2204] upgrade Quarkus to 3.27.2 and Spring Boot to 3.5.10 [incubator-kie-optaplanner]
yesamer commented on code in PR #3201: URL: https://github.com/apache/incubator-kie-optaplanner/pull/3201#discussion_r2754616864 ## build/optaplanner-build-parent/pom.xml: ## @@ -46,20 +46,20 @@ 2.22.1 1.4.21 6.6.7 -3.20.3 -4.1.127.Final +3.27.2 +4.1.130.Final 3.6.1 1.11.0 Review Comment: IN drools you updated `common.text` to 1.14.0. Please keep consistency, when possible. -- 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] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [DO-NOT-MERGE][incubator-kie-issues-2204] upgrade Quarkus to 3.27.2 and Spring Boot to 3.5.10 [incubator-kie-optaplanner]
yesamer commented on code in PR #3201: URL: https://github.com/apache/incubator-kie-optaplanner/pull/3201#discussion_r2754616864 ## build/optaplanner-build-parent/pom.xml: ## @@ -46,20 +46,20 @@ 2.22.1 1.4.21 6.6.7 -3.20.3 -4.1.127.Final +3.27.2 +4.1.130.Final 3.6.1 1.11.0 Review Comment: IN drools yoy updated common.text to 1.14.0. Please keep consistency, when possible. -- 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] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [DO-NOT-MERGE][incubator-kie-issues-2204] upgrade Quarkus to 3.27.2 and Spring Boot to 3.5.10 [incubator-kie-optaplanner]
jeejz commented on code in PR #3201: URL: https://github.com/apache/incubator-kie-optaplanner/pull/3201#discussion_r2744873388 ## build/optaplanner-build-parent/pom.xml: ## @@ -46,20 +46,20 @@ 2.22.1 1.4.21 6.6.7 -3.20.3 -4.1.127.Final +3.27.2 +4.1.130.Final 3.6.1 1.11.0 4.0.0 5.4.1 -3.27.7 -2.3.32 +3.27.6 Review Comment: Thanks @yesamer , that version change is made. -- 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] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] [DO-NOT-MERGE][incubator-kie-issues-2204] upgrade Quarkus to 3.27.2 and Spring Boot to 3.5.10 [incubator-kie-optaplanner]
yesamer commented on code in PR #3201: URL: https://github.com/apache/incubator-kie-optaplanner/pull/3201#discussion_r2741542401 ## build/optaplanner-build-parent/pom.xml: ## @@ -46,20 +46,20 @@ 2.22.1 1.4.21 6.6.7 -3.20.3 -4.1.127.Final +3.27.2 +4.1.130.Final 3.6.1 1.11.0 4.0.0 5.4.1 -3.27.7 -2.3.32 +3.27.6 Review Comment: @jeejz That assertJ version is affected by a CVE, https://github.com/apache/incubator-kie-drools/security/dependabot/80 - I suggest using .7 -- 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] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
