Skip to content

sbt: upgrade dependency to v1.12.9/v2.0.0-RC11#436

Merged
kitbellew merged 2 commits intoscalameta:mainfrom
kitbellew:436
Apr 10, 2026
Merged

sbt: upgrade dependency to v1.12.9/v2.0.0-RC11#436
kitbellew merged 2 commits intoscalameta:mainfrom
kitbellew:436

Conversation

@kitbellew
Copy link
Copy Markdown
Contributor

@kitbellew kitbellew commented Apr 8, 2026

That version fixes ability to download scala-library at given version, rather than the scala version of the build.

See sbt/sbt#9028 (2.x) and coursier/sbt-coursier#609 (1.x). Helps with #433.

That version fixes ability to download scala-library at given version,
rather than the scala version of the build.

See sbt/sbt#9028 (2.x) and coursier/sbt-coursier#609 (1.x).
@kitbellew kitbellew changed the title sbt: downgrade dependency for 2.12 to 1.2.8 sbt: upgrade dependency to v1.12.9/v2.0.0-RC11 Apr 8, 2026
@kitbellew kitbellew requested a review from tgodzik April 8, 2026 22:15
Comment thread build.sbt
Copy link
Copy Markdown
Contributor

@tgodzik tgodzik left a comment

Choose a reason for hiding this comment

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

just one suggestion otherwise LGTM

if (mintags.isEmpty) curtags.nonEmpty
else curtags.isEmpty || cmp(mintags, curtags) > 0
}
if (bad) sys.error(s"sbt-scalafmt requires sbt $v+ [current=$sbtVersion")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

MAybe warning would be enough? But that's up to you

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

if the version is older, download will fall with likely cryptic explanation. they might not see the warning just above.

@kitbellew kitbellew merged commit f5f85aa into scalameta:main Apr 10, 2026
4 checks passed
@kitbellew kitbellew deleted the 436 branch April 10, 2026 14:56
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