Skip to content

build issue in nightly-builds.yml#1329

Merged
He-Pin merged 2 commits intomainfrom
pjfanning-patch-1
May 17, 2024
Merged

build issue in nightly-builds.yml#1329
He-Pin merged 2 commits intomainfrom
pjfanning-patch-1

Conversation

@pjfanning
Copy link
Member

@pjfanning pjfanning commented May 17, 2024

relates to #1324

See failures in JDK11 builds in https://github.com/apache/pekko/actions/runs/9104380012/job/25028122890

The affected sbt calls only happen when Java version is 11 due to conditions in the workflow yml.

sbt \
-Dpekko.genjavadoc.enabled=${{ matrix.scalaVersion != '3.3' }} \
"+~ ${{ matrix.scalaVersion }} doc"
-Dpekko.genjavadoc.enabled=${{ matrix.scalaVersion.startsWith("3") }} \
Copy link
Member Author

Choose a reason for hiding this comment

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

#1324 changed the matrix scalaVersions to be of the form 3.3.x. I think it is safer not to rely exact matches here.

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

@He-Pin He-Pin left a comment

Choose a reason for hiding this comment

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

Lgtm

@He-Pin He-Pin merged commit c84f1b2 into main May 17, 2024
@He-Pin He-Pin deleted the pjfanning-patch-1 branch May 17, 2024 04:35
@He-Pin
Copy link
Member

He-Pin commented May 17, 2024

The workflow is not valid. .github/workflows/nightly-builds.yml (Line: 177, Col: 14): Unexpected symbol: '('. Located at position 31 within expression: matrix.scalaVersion.startsWith("3") --

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