-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Don't use custom serialization when reporting type validation problems #29189
base: master
Are you sure you want to change the base?
Conversation
if (serviceClass.isAssignableFrom(InternalProblems.class)) { | ||
return allServices.find(InternalProblems.class); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for looking into this @hegyibalint!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trivial solution, isn't it?!... 😂
5e85e4b
to
ed36a1d
Compare
344f636
to
0faa7f2
Compare
I've triggered the following builds for you. Click here to see all build failures. |
1518456
to
ccbf9c3
Compare
I've triggered the following builds for you. Click here to see all build failures. |
@bot-gradle test this |
I've triggered the following builds for you. Click here to see all build failures. |
api(project(":build-operations")) | ||
api(projects.serviceProvider) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SanityCheck failed wo/this change.
] | ||
} | ||
|
||
verifyAll(receivedProblem(0)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I only removed the if
statement.
@@ -16,6 +16,6 @@ | |||
|
|||
package org.gradle.plugin.devel.tasks | |||
|
|||
class CommonPluginValidationIntegrationTest extends AbstractPluginValidationIntegrationSpec implements ValitdatePluginsTrait { | |||
class CommonPluginValidationIntegrationTest extends AbstractPluginValidationIntegrationSpec implements ValidatePluginsTrait { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo fixed.
Fixes #29188