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 scala and sbt versions for scala-steward #27

Merged
merged 1 commit into from
Aug 1, 2024

Conversation

emdash-ie
Copy link
Contributor

@emdash-ie emdash-ie commented Aug 1, 2024

This repository recently caused scala-steward to fail, because it doesn’t currently meet the minimum required sbt and scala versions for scala steward to work. (Those versions are outlined in this issue.

This PR updates the sbt version in use to the newest versions and the two scala versions in use to the latest patch versions for those minor versions.

This should be sufficient to let scala-steward run on this repository: I have managed to successfully run sbt +test locally with these changes using Java 21.

This repository recently caused scala-steward to fail, because it
doesn’t currently meet the minimum required sbt and scala versions for
scala steward to work. (Those versions are outlined in [this
issue](guardian/maintaining-scala-projects#7).

This commit updates the sbt version in use to the newest versions and
the two scala versions in use to the latest patch versions for those
minor versions.

This should be sufficient to let scala-steward run on this repository: I
have managed to successfully run `sbt +test` locally with these changes
using Java 21.
@emdash-ie emdash-ie requested review from rtyley and AshCorr August 1, 2024 12:49
@emdash-ie
Copy link
Contributor Author

I’m not super familiar with this repository and who owns it, so I have requested review from two recent contributors!

@rtyley
Copy link
Member

rtyley commented Aug 1, 2024

I’m not super familiar with this repository and who owns it

I like to look to see which GitHub team is an 'admin' in the settings (this might not be accessible to you if you're not an admin on this repo):

image

In this case, @guardian/identity is the only admin, so I guess they're the owners? This would also make sense because these are the only repos using this library so far as I can tell from GitHub search:

Copy link
Member

@rtyley rtyley left a comment

Choose a reason for hiding this comment

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

This looks great to me! As mentioned in guardian/maintaining-scala-projects#7, we need to be on these minimum versions to allow compiling with Scala 21:

  • Scala: 2.12.18+, 2.13.11+, 3.3.1+
  • sbt: 1.9.0+

@emdash-ie
Copy link
Contributor Author

I’m not super familiar with this repository and who owns it

I like to look to see which GitHub team is an 'admin' in the settings (this might not be accessible to you if you're not an admin on this repo):
image

Thanks for the suggestion, I didn’t think to check that. It looks like I don’t have access fot this repository, alright.

@emdash-ie emdash-ie merged commit eddfb81 into main Aug 1, 2024
1 check passed
@emdash-ie emdash-ie deleted the update-scala-versions-for-scala-steward branch August 1, 2024 14:27
@rtyley
Copy link
Member

rtyley commented Aug 1, 2024

I can confirm that's fixed the Scala Steward workflow:

image

Scala Steward has now raised these for this repo:

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