Skip to content

TA-4413: Add sonarcloud integration#282

Merged
Zgjim Haziri (ZgjimHaziri) merged 1 commit into
masterfrom
TA-4413-add-sonarcloud-in-content-cli
Nov 18, 2025
Merged

TA-4413: Add sonarcloud integration#282
Zgjim Haziri (ZgjimHaziri) merged 1 commit into
masterfrom
TA-4413-add-sonarcloud-in-content-cli

Conversation

@ZgjimHaziri

Copy link
Copy Markdown
Contributor

Description

  • Added properties file which sets up the sonar project configs.
  • Added a workflow which does the Sonarcloud scan on the PR.
  • After the PR is merged, will add the Sonarcloud check scan as a required passed action in the PR build.

Note:

  • The Sonarcloud project of Content CLI is private, so besides the report generation of the PR here, it cannot be accesed by non-Celonis contributors.

Relevant links

Checklist

  • I have self-reviewed this PR
  • I have tested the change and proved that it works in different scenarios
  • I have updated docs if needed

@ZgjimHaziri Zgjim Haziri (ZgjimHaziri) requested a review from a team as a code owner November 17, 2025 13:02
@sonarqubecloud

Copy link
Copy Markdown

@github-advanced-security

Copy link
Copy Markdown

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@ZgjimHaziri Zgjim Haziri (ZgjimHaziri) merged commit c3b6ca0 into master Nov 18, 2025
4 checks passed
@ZgjimHaziri Zgjim Haziri (ZgjimHaziri) deleted the TA-4413-add-sonarcloud-in-content-cli branch November 18, 2025 10:04
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.

7 participants