New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #24768: Update Scala dependencies #5624
Fixes #24768: Update Scala dependencies #5624
Conversation
PR updated with a new commit |
<fs2-version>3.10.2</fs2-version> | ||
<shapeless-version>2.3.10</shapeless-version> | ||
<cats-effect-version>3.5.2</cats-effect-version> | ||
<dev-zio-version>2.0.19</dev-zio-version> | ||
<zio-cats-version>23.1.0.0</zio-cats-version> <!-- gives fs2 3.1.6, but doobie 1.0.0-RC1 is in 3.0.3 --> | ||
<cats-effect-version>3.5.4</cats-effect-version> | ||
<dev-zio-version>2.0.22</dev-zio-version> | ||
<zio-cats-version>23.1.0.2</zio-cats-version> <!-- gives fs2 3.10.2, but doobie 1.0.0-RC5 is in 3.9.3 --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fs2 : 3.3.0 -> 3.10.2 the update of the library does seem to have been forgotten, although cats-effect specifies it needs fs2 3.7.0+ since CE 3.5.0. As far as we know this has been working fine since we updated to 3.5.0 though...
PR updated with a new commit |
webapp/sources/pom.xml
Outdated
@@ -422,7 +422,7 @@ limitations under the License. | |||
<rudder-major-version>8.2</rudder-major-version> | |||
<rudder-version>8.2.0~alpha1-SNAPSHOT</rudder-version> | |||
|
|||
<scala-version>2.13.12</scala-version> | |||
<scala-version>2.13.13</scala-version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should not have any issue since we use -Xsource:3-cross
when migrating to scala 3
this PR is the one which will migrate to scala 2.13.13, with all needed rewrites : https://github.com/Normation/rudder/pull/5548/files
PR updated with a new commit |
1 similar comment
PR updated with a new commit |
This PR is not mergeable to upper versions. |
OK, squash merging this PR |
fee6a43
to
d7bf321
Compare
https://issues.rudder.io/issues/24768