mbien commented on code in PR #337: URL: https://github.com/apache/maven-indexer/pull/337#discussion_r1421508426
########## pom.xml: ########## @@ -106,9 +105,9 @@ under the License. <eclipse-sisu.version>0.9.0.M2</eclipse-sisu.version> <guice.version>6.0.0</guice.version> - <lucene.version>9.8.0</lucene.version> + <lucene.version>9.9.0</lucene.version> <maven.version>3.9.5</maven.version> Review Comment: roger, will update and force push in a sec ########## pom.xml: ########## @@ -268,7 +267,7 @@ under the License. <dependency> <groupId>org.eclipse.jetty</groupId> <artifactId>jetty-webapp</artifactId> - <version>9.4.51.v20230217</version> + <version>9.4.53.v20231009</version> Review Comment: yep. the only reason I didn't change is because I know from running my old blog on a rapsi, that Jetty 10.x changed quite a few tings internally (module system etc and even some property names of the plugin) which can break logging for example. So I didn't want to blindly bump the major version without knowing the usages very well. -- 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: issues-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org