Skip to content

Add a Scala 3.9 nightly CI job - #27043

Merged
Gedochao merged 1 commit into
scala:mainfrom
Gedochao:feature/3.9-nightly
Sep 10, 2026
Merged

Gedochao merged 1 commit into
scala:mainfrom
Gedochao:feature/3.9-nightly

Conversation

@Gedochao

@Gedochao Gedochao commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

This adds a Scala 3.9 LTS nightlies job, to mirror the Scala 3.3 LTS one.

Have you relied on LLM-based tools in this contribution?

Yes - but it's hard to test it other than yml syntax verification

How was the solution tested?

It wasn't

@SolalPirelli

Copy link
Copy Markdown
Contributor

What's the reason for it being different from the 3.3 one? I kind of expected it to use a matrix for this

@Gedochao

Copy link
Copy Markdown
Contributor Author

It uses different Java and PGP, which I assumed makes them different enough to warrant separate jobs

@tgodzik

tgodzik commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

What's the reason for it being different from the 3.3 one? I kind of expected it to use a matrix for this

The build changed heavily since 3.3 and I was unable to port the changes. Also not worth doing the changes to build separately.

@SolalPirelli SolalPirelli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I still think there should be a way to reuse the same code and invoke it with different (java version, pgp key) tuples but you'll be the one maintaining this so as you wish ;P

@tgodzik tgodzik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Gedochao
Gedochao merged commit a66ca79 into scala:main Sep 10, 2026
63 checks passed
@Gedochao
Gedochao deleted the feature/3.9-nightly branch September 10, 2026 11:54
Gedochao added a commit that referenced this pull request Sep 11, 2026
Well, that's awkward.
This is a follow-up fix to #27043 

```
error parsing called workflow
".github/workflows/release-nightly.yml"
-> "./.github/workflows/release-maven-artifacts.yml" (source branch with sha:1fdff21b7cf7511564b420c277d3755a80288948)
: (Line: 53, Col: 3): The identifier 'release-maven-lts-3.3-artifacts' is invalid. IDs may only contain alphanumeric characters, '_', and '-'. IDs must start with a letter or '_' and must be less than 100 characters., (Line: 104, Col: 3): The identifier 'release-maven-lts-3.9-artifacts' is invalid. IDs may only contain alphanumeric characters, '_', and '-'. IDs must start with a letter or '_' and must be less than 100 characters.
```

## Have you relied on LLM-based tools in this contribution?
No

## How was the solution tested?
Non-code change, no tests needed
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.

3 participants