-
Notifications
You must be signed in to change notification settings - Fork 668
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
SONARJAVA-4988: Use SonarLintCache component and make it accessible to custom rules via the caching APIs #4792
Conversation
0bbf25e
to
3641961
Compare
DBD custom rules need this information to turn off saving IR to the filesystem in a SonarLint context
3641961
to
686e380
Compare
0a52aa2
to
6da8f93
Compare
issue.setPrimaryLocation((InputFile) fileOrProject, analyzerMessage.getMessage(), textSpan.startLine, textSpan.startCharacter, | ||
textSpan.endLine, textSpan.endCharacter); |
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.
Nothing changed here, this is just the auto formatting.
"Cannot determine whether the context allows skipping unchanged files: canSkipUnchangedFiles not part of sonar-plugin-api. " + | ||
"Not skipping. {}", |
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.
Again, this is just a formatting change.
6da8f93
to
5c00fc8
Compare
private static final String LOG_MESSAGE_FILES_CAN_BE_SKIPPED = | ||
"The Java analyzer is running in a context where unchanged files can be skipped. " + | ||
"Full analysis is performed for changed files, optimized analysis for unchanged files."; | ||
private static final String LOG_MESSAGE_FILES_CANNOT_BE_SKIPPED = | ||
"The Java analyzer cannot skip unchanged files in this context. A full analysis is performed for all files."; | ||
private static final String LOG_MESSAGE_CANNOT_DETERMINE_IF_FILES_CAN_BE_SKIPPED = | ||
"Cannot determine whether the context allows skipping unchanged files: " + | ||
"canSkipUnchangedFiles not part of sonar-plugin-api. Not skipping. {}"; |
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.
These are again just formatting changes.
5c00fc8
to
9dc281f
Compare
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, good work in the comments to explain why we have this temporary hack, it should help later to remove it.
|
Please ensure your pull request adheres to the following guidelines: