Skip to content

Commit 09e0a90

Browse files
Fix Ruff linting errors
- Remove unnecessary else after break (RET508) - Add trailing commas (COM812) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent 78a5b74 commit 09e0a90

File tree

2 files changed

+8
-9
lines changed

2 files changed

+8
-9
lines changed

dojo/tools/api_sonarqube/api_client.py

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -282,13 +282,12 @@ def find_sca_risks(self, component_key, organization=None, branch=None):
282282
# Flat array response (no pagination metadata)
283283
risks.extend(response_data)
284284
break
285-
else:
286-
# Paginated response with issuesReleases array
287-
risks_page = response_data.get("issuesReleases", [])
288-
if not risks_page:
289-
break
290-
risks.extend(risks_page)
291-
page += 1
285+
# Paginated response with issuesReleases array
286+
risks_page = response_data.get("issuesReleases", [])
287+
if not risks_page:
288+
break
289+
risks.extend(risks_page)
290+
page += 1
292291

293292
return risks
294293

unittests/tools/test_api_sonarqube_importer.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,9 @@ def dummy_sca_risks(self, *args, **kwargs):
6161
"page": {
6262
"pageIndex": 1,
6363
"pageSize": 500,
64-
"total": len(risks)
64+
"total": len(risks),
6565
},
66-
"issuesReleases": risks
66+
"issuesReleases": risks,
6767
}
6868

6969

0 commit comments

Comments
 (0)