Skip to content

add scala-java8-compat and jackson-module-scala to community build #13168

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

Merged
merged 2 commits into from
Aug 5, 2021
Merged

add scala-java8-compat and jackson-module-scala to community build #13168

merged 2 commits into from
Aug 5, 2021

Conversation

pjfanning
Copy link
Contributor

@pjfanning pjfanning commented Jul 27, 2021

I tried to follow the steps in https://github.com/lampepfl/dotty/tree/master/community-build - feel free to request changes

@pjfanning pjfanning marked this pull request as draft July 27, 2021 13:17
Copy link
Contributor

@griggt griggt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems that this has a binary dependency on the Scala 3 artifact scala-java8-compat_3-1.0.0.jar

The projects in the community build normally build all their Scala 3 dependencies from source, which would mean adding scala-java8-compat to the community build together here with jackson-module-scala.

wdyt @anatoliykmetyuk

@anatoliykmetyuk
Copy link
Contributor

Yes, it would be good not to break the existing convention and also include the scala-java8-compat in this PR.

@pjfanning pjfanning marked this pull request as ready for review August 2, 2021 09:03
@pjfanning
Copy link
Contributor Author

@anatoliykmetyuk @griggt I've added scala-java8-compat to this PR

@@ -792,6 +805,7 @@ def allProjects = List(
projects.scalacheckEffect,
projects.fs2,
projects.libretto,
projects.jacksonModuleScala,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why isn't the java8-compat included in this list?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

apologies - missed that, I've fixed up the PR to include that

add scala-java8-compat

add scalaJava8Compat
@pjfanning pjfanning changed the title Draft: add jackson-module-scala to community build add scala-java8-compat and jackson-module-scala to community build Aug 2, 2021
@pjfanning
Copy link
Contributor Author

the CI build is failing due to org.scala-lang:scala-compiler:3.0.3-RC1-bin-SNAPSHOT and other related jars not being found

@griggt
Copy link
Contributor

griggt commented Aug 2, 2021

the CI build is failing due to org.scala-lang:scala-compiler:3.0.3-RC1-bin-SNAPSHOT and other related jars not being found

I've pushed a workaround

@anatoliykmetyuk anatoliykmetyuk merged commit 7d702a3 into scala:master Aug 5, 2021
@pjfanning pjfanning deleted the jackson-module-scala branch August 5, 2021 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants