-
Notifications
You must be signed in to change notification settings - Fork 49
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
SLCORE-777 Perform issue tracking before raising issue #952
SLCORE-777 Perform issue tracking before raising issue #952
Conversation
cd0e846
to
f56be07
Compare
93bae13
to
375a1d2
Compare
f56be07
to
3af7a5d
Compare
a8f5a55
to
3628b3a
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.
Some minor comments, else LGTM
@@ -67,4 +82,16 @@ public Set<String> getReportedRuleKeys() { | |||
public Set<SonarLanguage> getDetectedLanguages() { | |||
return detectedLanguages; | |||
} | |||
|
|||
public List<RawIssue> getIssues() { |
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.
Maybe getRawIssues
, to make things explicit? Same for the field name
return issues; | ||
} | ||
|
||
public boolean isEnableTracking() { |
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.
Maybe isTrackingEnabled
?
private void streamIssue(String configScopeId, UUID analysisId, Issue issue, ConcurrentHashMap<String, GetRuleDetailsResponse> ruleDetailsCache, | ||
HashSet<String> reportedRuleKeys) { | ||
private void streamIssue(String configScopeId, UUID analysisId, Issue issue, ConcurrentHashMap<String, RuleDetailsForAnalysis> ruleDetailsCache, | ||
List<RawIssue> raisedIssues) { |
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 think this should be called rawIssues
. Raw issues are not necessarily raised, as raised to me means that we would display them to the user. In fact raiseIssues
could even be an alternative name for publishIssues
*/ | ||
@Deprecated(since = "10.2") |
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.
If we deprecate this one, we can also deprecate didRaiseIssues
for the client
bc713a3
to
ba61e79
Compare
|
6ab2e0c
into
feature/analysis-tracking
* Add medium tests for issue tracking * SLCORE-777 Perform issue tracking before raising issue --------- Co-authored-by: Damien Urruty <damien.urruty@sonarsource.com>
* Add medium tests for issue tracking * SLCORE-777 Perform issue tracking before raising issue --------- Co-authored-by: Damien Urruty <damien.urruty@sonarsource.com>
* Add medium tests for issue tracking * SLCORE-777 Perform issue tracking before raising issue --------- Co-authored-by: Damien Urruty <damien.urruty@sonarsource.com>
No description provided.