Skip to content

build: Prepare Scala 3 build #590

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

Closed
wants to merge 3 commits into from
Closed

build: Prepare Scala 3 build #590

wants to merge 3 commits into from

Conversation

jakubjanecek
Copy link
Collaborator

@jakubjanecek jakubjanecek commented Jun 10, 2021

Preparation for Scala 3 build.

We are missing stable build of http4s for Scala 3 and we are missing build of PureConfig with automatic derivations (currently the supporrt is only experimental).

There are still some issues in Scala 3 compilation which need to be solved.

@jakubjanecek
Copy link
Collaborator Author

jakubjanecek commented Sep 24, 2021

@jendakol @sideeffffect I realized that we could probably try to finish this PR. PureConfig already has support for Scala 3 but not for ConfigWriter so I decided to remove those few ConfigWriter instances we have (I don't want them to block this PR, their not really needed for SST, they are a useful thing for someone who wants to write out configuration somewhere, that can be done in the app directly if needed).

But the problem is that PureConfig does not want to derive ConfigReader instances for me and I don't really know how to solve it. Do you have any experience with this? Could you have a look at it?

It's not really documented anywhere but the only thing I can think of is that maybe we will need to split the code and have different derivation for 2.x and 3.x but I've never done that before.

@tomasherman
Copy link
Contributor

replaced by #872

@tomasherman tomasherman closed this Apr 9, 2022
@jakubjanecek jakubjanecek deleted the feat/scala3 branch July 14, 2022 14:49
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.

2 participants