Skip to content

Integrate SonarQube static analysis tool#49

Open
IndyaG wants to merge 1 commit into
mainfrom
sonar-integ-test
Open

Integrate SonarQube static analysis tool#49
IndyaG wants to merge 1 commit into
mainfrom
sonar-integ-test

Conversation

@IndyaG
Copy link
Copy Markdown

@IndyaG IndyaG commented Oct 25, 2025

Tool Description

SonarQube is a tool that analyzes your code to find bugs, vulnerabilities, and code quality issues. It’s mainly used as a static analysis tool, meaning it checks your code without running it, and gives feedback on how to improve readability, maintainability, and security. (SonarQube is not on the starter list)

Changes Made

  • Added sonar-project.properties configuration file for SonarCloud.
  • Installed SonarScanner via npx sonar-scanner for project analysis.
  • Configured project key, organization, and source/test directories for analysis.
  • Added .scannerwork directory (generated by SonarScanner) to track scan artifacts.

Evidence of Successful Integration

  • SonarCloud scan executed successfully on branch sonar-integ-test.
  • Analysis output logged in sonarqube-output.txt

Notes

SonarQube has an interface to view the output of the result of the anaylsis on https://sonarcloud.io/ . Attached is the result of the scans on Sonar's dashboard
image

@IndyaG IndyaG self-assigned this Oct 25, 2025
@IndyaG
Copy link
Copy Markdown
Author

IndyaG commented Oct 25, 2025

The SonarQube integration passes locally but fails on GitHub Actions. I attempted to fix the issue, but it continues to fail. Since we are currently evaluating this tool to determine whether we want to use it, this should be considered when deciding whether to adopt it.

Copy link
Copy Markdown

@dkhalifa25 dkhalifa25 left a comment

Choose a reason for hiding this comment

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

lgtm!!

Copy link
Copy Markdown

@nniknam1 nniknam1 left a comment

Choose a reason for hiding this comment

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

Your PR lgtm and now it also passes on Github Actions too.

Copy link
Copy Markdown

@haolinwa haolinwa left a comment

Choose a reason for hiding this comment

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

lgtm

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.

4 participants