Skip to content

BUILD-8715 Add sonar platforms support for all actions#39

Merged
mikolaj-matuszny-ext-sonarsource merged 1 commit intomasterfrom
feat/mmatuszny/BUILD-8715
Aug 5, 2025
Merged

BUILD-8715 Add sonar platforms support for all actions#39
mikolaj-matuszny-ext-sonarsource merged 1 commit intomasterfrom
feat/mmatuszny/BUILD-8715

Conversation

@mikolaj-matuszny-ext-sonarsource
Copy link
Copy Markdown
Contributor

@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource commented Aug 4, 2025

BUILD-8715

Adds a choice for sonar-platform when using actions

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Aug 5, 2025

🤖 Pull Request summary

Adds configurable SonarQube platform selection to build actions.

New input parameter: Added sonar-platform input (next/sqc-eu/sqc-us) with "next" as default across all three build actions
Dynamic vault path resolution: Replaced hardcoded "next" vault paths with conditional logic mapping platform values to vault keys (sqc-eu→sonarcloud, sqc-us→sonarqube-us, next→next)
Poetry action enhancement: Added SonarQube environment variables (currently commented as disabled in build script)

Review focus: The nested ternary expressions in vault secret paths are complex and error-prone - consider extracting to a separate step for better readability and testing.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource merged commit 6a7be67 into master Aug 5, 2025
8 checks passed
@mikolaj-matuszny-ext-sonarsource mikolaj-matuszny-ext-sonarsource deleted the feat/mmatuszny/BUILD-8715 branch August 5, 2025 07:20
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.

2 participants