Update all CI for release of JDK 17 - Githubissues

scala / scala-dev

Scala 2 team issues. Not for user-facing bugs or directly actionable user-facing improvements. For build/test/infra and for longer-term planning and idea tracking. Our bug tracker is at https://github.com/scala/bug/issues

Apache License 2.0

130 stars 15 forks source link

Update all CI for release of JDK 17 #787

Closed SethTisue closed 2 years ago

SethTisue commented 2 years ago

wait for the release to be picked up by ~AdoptOpenJDK~ Temurin and by SDKMAN

then:

[x] community build
- https://github.com/scala/community-build/issues/1492
[x] Fortify
[ ] all our GitHub Actions configs will need 17-ea changed to 17
[x] make sure the Travis-CI configs using default.yml in this repo pick up the release
[ ] also look for opportunities to test on JDK 17 where we weren't before, since it's an LTS release. we already added it to many of our CI matrices but maybe not all.

(It's probably too early to bother adding 18-ea anywhere.)

SethTisue commented 2 years ago

relevant: https://github.com/sdkman/sdkman-db-migrations/issues/543

SethTisue commented 2 years ago

scala/scala#9772 and scala/scala#9773 and scala/scala#9770 take care of that repo

SethTisue commented 2 years ago

I haven't done every module repo yet, but I'm including 17 upgrades as I work through #785

SethTisue commented 2 years ago

I'm going to close this as I believe everything important is covered. we should keep an ongoing eye out for less important repos that may have been missed

SethTisue commented 2 years ago

https://github.com/lampepfl/dotty/pull/13629

SethTisue commented 2 years ago

https://github.com/scala/scala-library-next/pull/98#discussion_r720431701 says

Adopt OpenJDK got moved to Eclipse Temurin and won't be updated anymore. It is highly recommended to migrate workflows from adopt to temurin to keep receiving software and security updates

Thanks @griggt for calling this to my attention. I'll need to go around and update all the GitHub Actions configs.

SethTisue commented 2 years ago

I PRed the change in every repo I could think of.