Skip to content
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

Update sbt-typelevel to 0.5.3 in series/3.x #1247

Merged
merged 2 commits into from
Sep 29, 2023

Conversation

aartigao
Copy link
Contributor

@aartigao aartigao commented Sep 27, 2023

Bumping to latest 0.5.x version of sbt-typelevel (this PR supersedes #1244 which didn't compile 2.13 due to a bug).

As noted in v0.5.0 release notes:

[...] some subtle changes to type inference and implicit resolution [...]

I had to move the import

import fs2.kafka.internal.converters.collection._

inside asJavaMap to force the implicit resolution of asJava instead of trying to use the one in FoldableSyntax.

EDIT (29/09/2023)

For Scala 3.3.x, some compiler options were added that caused some compilation failures for warnings:

  • discarded non-Unit value of type StringBuilder
  • unused private member

Instead of trying to fix them here in this PR, I took the fast path to rollback to previous state and remove the offending compiler options. In the end, those warnings for Scala 3 have been there, hidden, in the previous versions without issues (that includes versions 2.12 and 2.13 too). I think it's best to address them in the future, in an eventual Scala 3 syntax migration, but there's a lot work to do before that.

@aartigao aartigao changed the title Update sbt-typelevel to 0.5.2 in series/3.x Update sbt-typelevel to 0.5.3 in series/3.x Sep 27, 2023
@aartigao aartigao merged commit ab37b19 into fd4s:series/3.x Sep 29, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant