Skip to content

set language version explicitly#915

Merged
trask merged 3 commits intoopen-telemetry:mainfrom
zeitlinger:set-language-version-explicitly
Nov 26, 2025
Merged

set language version explicitly#915
trask merged 3 commits intoopen-telemetry:mainfrom
zeitlinger:set-language-version-explicitly

Conversation

@zeitlinger
Copy link
Copy Markdown
Member

so that we can safely update JDK to latest version

@zeitlinger zeitlinger self-assigned this Nov 24, 2025
@zeitlinger zeitlinger requested review from a team as code owners November 24, 2025 11:51
@zeitlinger
Copy link
Copy Markdown
Member Author

@jaydeluca please check - you added those projects

Comment thread javaagent-declarative-configuration/build.gradle.kts
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>

java {
toolchain {
languageVersion.set(JavaLanguageVersion.of(17))
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why is Java 17 needed here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

it could be 8 here - I'll change

@trask trask merged commit 3e4521f into open-telemetry:main Nov 26, 2025
13 checks passed
@zeitlinger zeitlinger deleted the set-language-version-explicitly branch November 27, 2025 12:33
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