Skip to content

fix: sonarqube plugin api changes#42

Draft
psturc wants to merge 1 commit into
konflux-ci:mainfrom
psturc:KFLUXDP-714
Draft

fix: sonarqube plugin api changes#42
psturc wants to merge 1 commit into
konflux-ci:mainfrom
psturc:KFLUXDP-714

Conversation

@psturc

@psturc psturc commented Jan 15, 2026

Copy link
Copy Markdown
Member

necessary changes required to fetch projects from internal sonarqube instance using a regular user token

@snyk-io

snyk-io Bot commented Jan 15, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

kpiwko added a commit to kpiwko/devlake that referenced this pull request Mar 10, 2026
Add CSV-based e2e test for GitHub review reaction enrichment and a
mock HTTP server-based test for GitLab award emoji enrichment using
a pre-recorded fixture from gitlab.cee.redhat.com MR konflux-ci#42.

Fix three issues in enrich_gitlab_review_reactions.go:
- propagate cursor error instead of silently returning nil
- defer infoCursor.Close() to cover all exit paths
- use http.NewRequestWithContext to respect task cancellation

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant