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 version #9

Merged
merged 1 commit into from
Jan 26, 2024
Merged

Update sbt version #9

merged 1 commit into from
Jan 26, 2024

Conversation

mdedetrich
Copy link
Contributor

@mdedetrich mdedetrich commented Jan 26, 2024

Updates SBT to 1.9.7 (I opted for this instead of 1.9.8 because 1.9.8 has some known issues) while also getting rid of deprecated in syntax. Existing sbt scripted tests pass locally after this change (there aren't any standard tests).

After this PR is merged I will integrate sbt-github-actions so we can get some CI (will also remove travis CI).

@@ -2,6 +2,7 @@ lazy val `sbt-pull-request-validator` = (project in file("."))
.enablePlugins(AutomateHeaderPlugin)

sbtPlugin := true
enablePlugins(SbtPlugin)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is one of the few source breaking changes introduced in sbt over the years, specifically sbt 1.2.8.

I had to do a similar change when updating sbt version in multi-release-jar

@mdedetrich mdedetrich merged commit d8dd6f0 into sbt:main Jan 26, 2024
1 check passed
@mdedetrich mdedetrich deleted the update-sbt branch January 26, 2024 05:32
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.

2 participants