Skip to content

ci: fix Scoverage report path search depth #40

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.

Sign up for GitHub

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

Open
rabestro wants to merge 3 commits into
base: main
Choose a base branch
from
Open

ci: fix Scoverage report path search depth #40

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
26de320
ci: fix Scoverage report path search depth
rabestro Jul 15, 2026
a548e74
delete space
rabestro Jul 15, 2026
dac412c
Merge branch 'main' into ci/39-fix-scoverage-path
rabestro Jul 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 17 additions & 17 deletions .github/workflows/ci.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,23 +56,23 @@ jobs:
GITHUB_ACTOR: ${{ github.actor }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

- name: Locate Scoverage report
# The report path embeds the Scala version (target/out/jvm/scala-<ver>/...), so a
# hardcoded path in sonar-project.properties silently goes stale on every
# scalaVersion bump (the Scoverage sensor only WARNs, never fails, on a
# missing report — see dicechess-engine-scala#448 for how this bit that repo).
# Discover the real path at scan time and fail loudly if it's missing, instead
# of letting SonarCloud quietly show zero coverage.
if: ${{ env.SONAR_TOKEN != '' && github.actor != 'dependabot[bot]' }}
id: scoverage
run: |
REPORT=$(find target -maxdepth 10 -path '*/scoverage-report/scoverage.xml' -print -quit)
if [ -z "$REPORT" ]; then
echo "::error::No scoverage.xml found under target/ — SonarQube coverage import would silently fail."
exit 1
fi
echo "Found Scoverage report at $REPORT"
echo "report_path=$REPORT" >> "$GITHUB_OUTPUT"
- name: Locate Scoverage report
# The report path embeds the Scala version (target/out/jvm/scala-<ver>/...), so a
# hardcoded path in sonar-project.properties silently goes stale on every
# scalaVersion bump (the Scoverage sensor only WARNs, never fails, on a
# missing report — see dicechess-engine-scala#448 for how this bit that repo).
# Discover the real path at scan time and fail loudly if it's missing, instead
# of letting SonarCloud quietly show zero coverage.
if: ${{ env.SONAR_TOKEN != '' && github.actor != 'dependabot[bot]' }}
id: scoverage
run: |
REPORT=$(find target -maxdepth 10 -path '*/scoverage-report/scoverage.xml' -print -quit)
if [ -z "$REPORT" ]; then
echo "::error::No scoverage.xml found under target/ — SonarQube coverage import would silently fail."
exit 1
fi
echo "Found Scoverage report at $REPORT"
echo "report_path=$REPORT" >> "$GITHUB_OUTPUT"

# Imports the scoverage report (see sonar-project.properties) into SonarCloud.
# Requires Automatic Analysis to be OFF for this project (CI-based analysis instead).
Expand Down
Loading