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

Fix release so that it does not attempt to use Scala 2.12 #320

Conversation

rtyley
Copy link
Member

@rtyley rtyley commented Aug 2, 2024

After #318 dropped support for Scala 2.12, release 8.0.1 correctly no longer published Scala 2.12 artifacts, but unfortunately the 'root' project (which does not have any code!) was still defaulting to using Scala 2.12, because that is the default sbt uses if no scalaVersion is set.

The next time we tried to do a release (#319 (comment)) the automated-compatibility checking failed, because it saw that one project was using Scala 2.12, and so it attempted to download all the non-existent Scala 2.12 artifacts for release 8.0.11 - and failed.

To fix this, we can simply ensure that all modules in this sbt build default to Scala 2.13, not 2.12 - ie particularly, the 'root' module!

Testing

We ran a preview release on this PR, and the automated-compatibility checking succeeded 👍 :

image

Footnotes

  1. This behaviour seems like a bug in sbt-version-policy?! sbt-version-policy is already aware of the publish / skip := true flag, so should know that root does not have any artifacts...

After #318 dropped
support for Scala 2.12, release 8.0.1 correctly no longer published Scala 2.12
artifacts, but unfortunately the 'root' project (which does not have any
code!) was still defaulting to using Scala 2.12, because that is the
default sbt uses if no `scalaVersion` is set.

The next time we tried to do a release (#319 (comment))
the automated-compatibility checking failed, because it saw that one project
was using Scala 2.12, and so it attempted to download all the non-existent
Scala 2.12 artifacts for release 8.0.1 - and failed.

To fix this, we can simply ensure that all modules in this sbt build default
to Scala 2.13, not 2.12 - ie particularly, the 'root' module!
@gu-scala-library-release
Copy link
Contributor

@rtyley has published a preview version of this PR with release workflow run #30, based on commit 6574ed3:

8.0.2-PREVIEW.fix-release-process-so-that-it-does-not-attempt-to-use-scala-212.2024-08-02T1015.6574ed36

Want to make another preview release?

Click 'Run workflow' in the GitHub UI, specifying the fix-release-process-so-that-it-does-not-attempt-to-use-scala-2.12 branch, or use the GitHub CLI command:

gh workflow run release.yml --ref fix-release-process-so-that-it-does-not-attempt-to-use-scala-2.12

Want to make a full release after this PR is merged?

Click 'Run workflow' in the GitHub UI, leaving the branch as the default, or use the GitHub CLI command:

gh workflow run release.yml

@rtyley rtyley marked this pull request as ready for review August 2, 2024 10:18
@rtyley rtyley requested a review from waisingyiu August 2, 2024 10:19
Copy link
Contributor

@waisingyiu waisingyiu left a comment

Choose a reason for hiding this comment

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

👍

@rtyley rtyley merged commit b259991 into main Aug 2, 2024
10 checks passed
@rtyley rtyley deleted the fix-release-process-so-that-it-does-not-attempt-to-use-scala-2.12 branch August 2, 2024 10:19
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