Skip to content

Draft: 431 configure sonarcloud ci based analysis#432

Merged
jondricek merged 4 commits into
developfrom
431-configure-sonarcloud-ci-based-analysis
Aug 11, 2023
Merged

Draft: 431 configure sonarcloud ci based analysis#432
jondricek merged 4 commits into
developfrom
431-configure-sonarcloud-ci-based-analysis

Conversation

@seansica

Copy link
Copy Markdown
Contributor

Description of what has changed

  • Renames GitHub Workflow build.yml to static-code-analysis.yml to be more descriptive and accurate
  • Adds the InsonusK/get-latest-release@v1.0.1 GitHub Action plugin to the static code analysis CI workflow and sets environment variable SONAR_PROJECT_VERSION to the corresponding latest release semver value. This is to ensure sonar.projectVersion is always set to the latest release version in SonarCloud.
  • Adds and modifies various CI-based analysis parameters to sonar-project.properties

Issues addressed by pull request

Closes #431

seansica added 2 commits May 23, 2023 11:27
* Add get-latest-release plugin to set SONAR_PROJECT_VERSION;
@seansica seansica requested a review from jondricek May 23, 2023 15:41
@seansica seansica self-assigned this May 23, 2023
@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@jondricek jondricek merged commit 97debe2 into develop Aug 11, 2023
@jondricek jondricek deleted the 431-configure-sonarcloud-ci-based-analysis branch August 11, 2023 16:17
@sonarqubecloud

Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
13.4% 13.4% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

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