Thanks to visit codestin.com
Credit goes to github.com

Skip to content

silence warning on JDK 20 #10360

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

Merged
merged 1 commit into from
Mar 31, 2023
Merged

Conversation

SethTisue
Copy link
Member

@SethTisue SethTisue commented Mar 30, 2023

as uncovered by recent merge of #10311

once merged, should fix the JDK 20 failure seen at https://github.com/scala/scala/actions/runs/4566043533/jobs/8058001307

as uncovered by recent merge of scala#10311
@SethTisue SethTisue requested a review from NthPortal as a code owner March 30, 2023 21:17
@scala-jenkins scala-jenkins added this to the 2.13.12 milestone Mar 30, 2023
@SethTisue SethTisue modified the milestones: 2.13.12, 2.13.11 Mar 30, 2023
@SethTisue SethTisue added the internal not resulting in user-visible changes (build changes, tests, internal cleanups) label Mar 30, 2023
@SethTisue SethTisue requested a review from som-snytt March 30, 2023 21:18
Copy link
Contributor

@som-snytt som-snytt left a comment

Choose a reason for hiding this comment

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

Aha!

Could have been, in both places, and in homage to soc

@deprecated("ThreadDeath is deprecated for removal", since="JDK 20")

@lrytz lrytz merged commit 7a659e0 into scala:2.13.x Mar 31, 2023
@som-snytt
Copy link
Contributor

but the current annotation, while not as good, is ok because it's just test code.

Why does github give me a button to delete Seth's branch? I know I can push to it with useful updates; probably they expected me to push my suggestion, and now they offer me this button as a way to vent my frustration.

(I'm watching Westworld the reboot, and this seems very Westworld.)

@SethTisue SethTisue deleted the silence-jdk20-warning branch March 31, 2023 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal not resulting in user-visible changes (build changes, tests, internal cleanups)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants