-
Notifications
You must be signed in to change notification settings - Fork 85
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
SONARPY-1800 NonCallableCallCheck migration: Use the display name of the new type model #1777
SONARPY-1800 NonCallableCallCheck migration: Use the display name of the new type model #1777
Conversation
d8d65e3
to
7db3a7e
Compare
…the new type model
7db3a7e
to
c077762
Compare
Quality Gate passedIssues Measures |
if (name != null) { | ||
return String.format("Fix this call; \"%s\"%s is not callable.", name, addTypeName(calleeType)); | ||
return String.format("Fix this call; \"%s\"%s is not callable.", name, addTypeName(typeV2)); |
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.
In currently used Java version String.format
could be simplified by the String.formatted
method:
"Fix this call; \"%s\"%s is not callable.".formatted(name, addTypeName(typeV2))
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.
LGTM, just one minor suggestion related to strings formatting (no need to review again, feel free to ignore)
de9a80f
into
rnd/type-inference-engine-specification
…the new type model (#1777)
…the new type model (#1777)
…the new type model (#1777)
…the new type model (#1777)
No description provided.