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

Fix #459: Upgrade protobuf to 3.7.0 to get rid of Java 9+ warnings #644

Merged
merged 1 commit into from
Mar 13, 2019

Conversation

smarter
Copy link
Contributor

@smarter smarter commented Mar 11, 2019

I have no idea if this is binary-compatible so fingers crossed.

I have no idea if this is binary-compatible so fingers crossed.
@typesafe-tools
Copy link

A validation involving this pull request is in progress...

@typesafe-tools
Copy link

The validator has checked the following projects, tested using dbuild, projects built on top of each other.

Project Reference Commit
sbt develop sbt/sbt@354f85c
zinc pull/644/head bc7a8f5
io develop sbt/io@0ae8142
librarymanagement develop sbt/librarymanagement@6d35f32
util develop sbt/util@48574c2

❌ The result is: FAILED
(restart)

@smarter
Copy link
Contributor Author

smarter commented Mar 11, 2019

Looks like sbt doesn't compile with zinc master, so hard to test things more.

@smarter smarter requested review from eed3si9n and jvican March 11, 2019 15:47
@eed3si9n
Copy link
Member

I saw the drone job fail first, so I restarted it. Now it's green, so it's good to go, I think.
There's some changes that were made in IO that needs to propagate out for the Dbuild validation.

@jvican jvican merged commit 841966f into sbt:develop Mar 13, 2019
@eed3si9n eed3si9n added this to the 1.3.0 milestone Apr 28, 2019
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.

4 participants