Skip to content

Update copyright, Scala Native, Scala, sbt, ubuntu and macOS to current versions #107

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

LeeTibbert
Copy link
Contributor

@LeeTibbert LeeTibbert commented Jun 3, 2025

A user in the Scala Native Discourse channel reported that these versions
were out of date and that Metals was recommending an update
to the latest Scala LTS version: Done.

Also removed some bit rot from CI versions.

I believe/hope that this will merge cleanly on top of pending PRs.

On a quick read, it looks like most of those will not be merged.
The scala-steward changes might mean I have to re-base after
them.

I will need to submit another PR once SN 0.5.8 had been released and aged
for a fortnight.

@LeeTibbert LeeTibbert changed the title Update CI os & other versions Update copyright, Scala Native, Scala, sbt, ubuntu and macOS to current versions Jun 3, 2025
@LeeTibbert
Copy link
Contributor Author

@ekrich

If you have a chance, could you take a look at the changes of this PR and see if you concur
or have recommendations? I've made a number of changes to os and JVM versions.
Seems to build.

I'd rest easier knowing that a second set of eyes had checked these. Thank you.

My central idea was to use, to the greatest extent possible, the versions, actions, and procedures
used in the Scala Native build itself.

Copy link
Member

@ekrich ekrich left a comment

Choose a reason for hiding this comment

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

The only thing I see is that most people are using adopt JVM.

Would something like this work for sbt?

steps:
    - uses: actions/checkout@v4
    - name: Setup Java
      uses: actions/setup-java@v4
      with:
        distribution: 'adopt'
        java-version: ${{ matrix.java }}
        cache: 'sbt'
    - uses: sbt/setup-sbt@v1
    - run: sbt +test

@LeeTibbert
Copy link
Contributor Author

LeeTibbert commented Jun 3, 2025

Eric,

Thank you for looking at this. I'll try again with 'adopt'.

That line in mainline was 'adopt'. I think I ran into a problem
so switched to what I know worked in SN CI. Always good
to re-establish a baseline.

Later: CI accepted 'adopt', great! One less change to break.

@LeeTibbert
Copy link
Contributor Author

@ekrich

This PR has been lagering for a while, its time has not come.

Do the sbt version and scala-native versions still make sense?
I am thinking of updating the former to the current 1.11.3 and the latter to 0.5.8.

Your suggestions and/or thoughts?

@ekrich
Copy link
Member

ekrich commented Jul 15, 2025

Sure, the latest bits are appropriate. Maybe ping Wojciech on this one.

@LeeTibbert
Copy link
Contributor Author

Sure, the latest bits are appropriate.

Thank you for the confirmation. I'll try to create some time this week for a proper
update. Busy week.

Maybe ping Wojciech on this one.

I wanted to make the code sensible (a.k.a current) given the elapsed time before
annoying Wojciech on this one.

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