-
Notifications
You must be signed in to change notification settings - Fork 101
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 to 2.13.12 #278
Conversation
@lefou sorry, but I merged the other PR and now there are quite some conflicts 😞 |
b019e90
to
7de7c0f
Compare
@cstamas I rebased this PR. It's ready for review and merge, although, I'd prefer if you could cut a |
So just to clarify:
Pls confirm |
Exactly that. 👍 |
Thank you! |
This PR updates from Scala 2.12 to 2.13. With the update to Scala 2.13.12, which is the latest available Scala 2.x version,
polyglot-scala
should also support Java 21.I suggest to have this PR and PR #277 (which updates from Scala 2.11 to 2.12) not in the same release, to give users of
polyglot-scala
a chance to migrate their builds step by step. But TBH, I have no clue how many users we still have.Along this update, I bumped various dependencies, migrated some Scala syntax and fixed deprecated or removed API call sites. I also fixed some issues with left-open file resources.