Copilot commented on code in PR #6410: URL: https://github.com/apache/texera/pull/6410#discussion_r3583797768
########## amber/build.sbt: ########## @@ -306,7 +306,7 @@ libraryDependencies += "com.esotericsoftware" % "kryo5" % "5.6.0" libraryDependencies += "io.altoo" %% "pekko-kryo-serialization" % "1.3.0" // https://mvnrepository.com/artifact/io.altoo/scala-kryo-serialization -libraryDependencies += "io.altoo" %% "scala-kryo-serialization" % "1.3.0" +libraryDependencies += "io.altoo" %% "scala-kryo-serialization" % "1.5.2" Review Comment: Bumping scala-kryo-serialization changes the bundled jar version, but amber/LICENSE-binary-java still lists io.altoo.scala-kryo-serialization_2.13-1.3.0.jar. CI runs bin/licensing/check_binary_deps.py against amber/LICENSE-binary-java for the unzipped dist lib dir, so this PR will fail the license drift check unless LICENSE-binary-java is updated to match the new resolved jar version. -- 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]
