Skip to content

Run SonarCloud analysis only when SONAR_TOKEN is available #195

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 6, 2020

Conversation

joschi
Copy link
Contributor

@joschi joschi commented Nov 3, 2020

Forked repositories don't have access to the SONAR_TOKEN secret and thus cannot run the SonarCloud analysis as part of the build.

Refs #184

@joschi joschi added the github_actions Pull requests that update Github_actions code label Nov 3, 2020
@joschi joschi self-assigned this Nov 3, 2020
Forked repositories don't have access to the `SONAR_TOKEN` secret and thus cannot run the SonarCloud analysis as part of the build.

Refs #184
@joschi joschi force-pushed the sonarcloud-source-repo branch from b8173bc to c01750f Compare December 6, 2020 21:10
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 6, 2020

Kudos, SonarCloud 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

@joschi joschi merged commit 471ed26 into master Dec 6, 2020
@joschi joschi deleted the sonarcloud-source-repo branch December 6, 2020 21:16
joschi added a commit that referenced this pull request Dec 6, 2020
Forked repositories don't have access to the `SONAR_TOKEN` secret and thus cannot run the SonarCloud analysis as part of the build.

Refs #184
@joschi joschi added this to the Release 2.0.0 milestone Dec 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update Github_actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant