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

Relax presence of Scala compiler warnings #23942

Merged
merged 1 commit into from Feb 17, 2023

Conversation

ljacomet
Copy link
Member

This is needed following the changes around handling implicit toolchains.

Issue #23905

This is needed following the changes around handling implicit
toolchains.

Issue #23905
@ljacomet ljacomet added this to the 8.0.1 milestone Feb 17, 2023
@ljacomet ljacomet requested a review from a team as a code owner February 17, 2023 14:06
@ljacomet ljacomet self-assigned this Feb 17, 2023
@ljacomet ljacomet requested a review from tresat February 17, 2023 14:06
@ljacomet
Copy link
Member Author

@bot-gradle test and merge

@bot-gradle
Copy link
Collaborator

Your PR is queued. See the queue page for details.

@jvandort
Copy link
Member

What warnings are getting output?

@bot-gradle
Copy link
Collaborator

I've triggered a build for you.

@ljacomet
Copy link
Member Author

Warnings are:

[Warn] : -target is deprecated: Use -release instead to compile against the correct platform API.

These come from changes done to fix #23905. A follow up is planned through #23943

@bot-gradle bot-gradle merged commit 68959bf into release Feb 17, 2023
@bot-gradle bot-gradle deleted the ljacomet/gradle8/scala-test-fix branch February 17, 2023 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants