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

Reply via email to