-
Notifications
You must be signed in to change notification settings - Fork 683
[compiler] Add Service.issueSeverity()
#6731
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
Conversation
✅ Docs preview has no changesThe preview was not built because there were no changes. Build ID: c7be8aa3cf5e3209fcecb761 |
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.
😍
public abstract fun getCodegenModels ()Ljava/lang/String; | ||
public abstract fun getDecapitalizeFields ()Ljava/lang/Boolean; | ||
public abstract fun getFailOnWarnings ()Ljava/lang/Boolean; | ||
public abstract fun getFieldsOnDisjointTypesMustMerge ()Ljava/lang/Boolean; |
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.
Did I just break the IJ plugin?
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.
Woops, yes indeed this is used by the plugin. I guess you can just return false instead, and deprecate it.
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.
Gotcha 👍
8a2c707
to
37f912f
Compare
Closes #6516
Deprecate
warnOnDeprecatedUsage
andfieldsOnDisjointTypesMustMerge
that can now be replaced by setting the desired severity.