Skip to content

Commit

Permalink
Merge pull request #2737 from bitshares/sonarcloud-configuration
Browse files Browse the repository at this point in the history
Remove SonarCloud cache and threads configuration as it is now by default
  • Loading branch information
abitmore authored Mar 25, 2023
2 parents dfd22b1 + 5d59c9e commit 70eece1
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/sonar-scan.yml
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,6 @@ jobs:
with:
path: |
ccache
sonar_cache
key: sonar-${{ env.OS_VERSION }}-${{ github.ref }}-${{ github.sha }}
restore-keys: |
sonar-${{ env.OS_VERSION }}-${{ github.ref }}-
Expand Down Expand Up @@ -185,7 +184,6 @@ jobs:
df -h
- name: Prepare for scanning with SonarScanner
run: |
mkdir -p sonar_cache
find _build/libraries/[acdenptuw]*/CMakeFiles/*.dir \
_build/libraries/plugins/*/CMakeFiles/*.dir \
-type d -print \
Expand Down
7 changes: 4 additions & 3 deletions sonar-project.properties
Original file line number Diff line number Diff line change
Expand Up @@ -12,14 +12,15 @@ sonar.links.ci=https://github.com/bitshares/bitshares-core/actions
sonar.links.issue=https://github.com/bitshares/bitshares-core/issues
sonar.links.scm=https://github.com/bitshares/bitshares-core/tree/master

# Note:
# According to docs, sonar.tests is ignored by the C/C++/Objective-C analyzer.
# See https://docs.sonarcloud.io/advanced-setup/languages/c-c-objective-c/#language-specific-properties
sonar.tests=tests

sonar.exclusions=programs/build_helper/**/*,libraries/fc/**/*,libraries/egenesis/egenesis_full.cpp
sonar.sources=libraries,programs
sonar.cfamily.build-wrapper-output=bw-output
sonar.cfamily.gcov.reportsPath=.
sonar.cfamily.threads=2
sonar.cfamily.cache.enabled=true
sonar.cfamily.cache.path=sonar_cache

# Decide which tree the current build belongs to in SonarCloud.
# Managed by the `set_sonar_branch*` script(s) when building with CI.
Expand Down

0 comments on commit 70eece1

Please sign in to comment.