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 to 1.5.2 #288

Closed
wants to merge 2 commits into from
Closed

Update sbt to 1.5.2 #288

wants to merge 2 commits into from

Conversation

felixbr
Copy link
Contributor

@felixbr felixbr commented May 24, 2021

Problem

sbt supports Scala 3 natively in the recent versions (e.g. 1.5.x) and I want the build to be ready for Scala 3.

Solution

Updating sbt from 1.3.10 to 1.5.2

Result

sbt is updated to 1.5.2 and most warnings addressed (i.e. auto-reloading, unused settings, and migration to slash-syntax for settings)

As usual, I cannot tell if this clashes with any Twitter-internal things, so please tell me if it does.

@felixbr felixbr mentioned this pull request May 24, 2021
9 tasks
@mosesn
Copy link
Contributor

mosesn commented Jun 2, 2021

@felixbr thanks for the PR! Sorry for the delay in getting back to you. I don't think this clashes with anything, let me double check.

Copy link
Contributor

@mosesn mosesn left a comment

Choose a reason for hiding this comment

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

From poking around, this looks good. I'm going to switch to 1.5.3 and try to apply your patches to scrooge / finagle / twitter-server / finatra, and hopefully we'll be able to merge this in soon-ish.

@mosesn
Copy link
Contributor

mosesn commented Jun 3, 2021

Sorry for the delay, I've run into a bunch of sbt weirdness. New sbt versions apparently rearrange the classpath. I'm hoping to have it sorted today or tomorrow, @eed3si9n has been tremendously helpful!

@mosesn
Copy link
Contributor

mosesn commented Jun 7, 2021

Thanks for your patience, this was merged in! 659e818

@mosesn mosesn closed this Jun 7, 2021
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