Skip to content
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

fix: update sonarqube issue collector by type/status/severity #6315

Merged
merged 4 commits into from
Oct 25, 2023

Conversation

abeizn
Copy link
Contributor

@abeizn abeizn commented Oct 24, 2023

Summary

update sonarqube issue collector by type/status/severity filter

Does this close any open issues?

Closes #6316

Screenshots

image

image

Other Information

Any other information that is important to this PR.

@abeizn abeizn requested a review from leric October 24, 2023 08:52
@abeizn abeizn force-pushed the update-sonarqube-issue-collector branch from ad6f84e to bc4f603 Compare October 24, 2023 10:08
@@ -127,13 +142,13 @@ func CollectIssues(taskCtx plugin.SubTaskContext) (err errors.Error) {
MidTime := time.Unix((createdAfterUnix+createdBeforeUnix)/2+1, 0)

// left part
iterator.Push(&SonarqubeIssueTimeIteratorNode{
iterator.Push(&SonarqubeIssueIteratorNode{
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

severity, type, status should be passed along?

@d4x1 d4x1 merged commit 34429ad into main Oct 25, 2023
10 checks passed
@d4x1 d4x1 deleted the update-sonarqube-issue-collector branch October 25, 2023 02:50
github-actions bot pushed a commit that referenced this pull request Oct 25, 2023
* fix: update sonarqube issue collector by type/status/severity

* fix: ci lint

* fix: add time filter

* fix: add filter condition on time filter
@github-actions
Copy link

🤖 cherry pick finished successfully 🎉!

@github-actions github-actions bot added the bot/auto-cherry-pick-completed auto cherry pick completed label Oct 25, 2023
d4x1 pushed a commit that referenced this pull request Oct 25, 2023
…#6324)

* fix: update sonarqube issue collector by type/status/severity

* fix: ci lint

* fix: add time filter

* fix: add filter condition on time filter

Co-authored-by: abeizn <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug][sonarqube] Cannot get more than 10000 results through web API
3 participants