Skip to content

Add basic coveralls.io support - #52

Open
Zain-Mahmoud wants to merge 16 commits into
mainfrom
add-coveralls
Open

Add basic coveralls.io support#52
Zain-Mahmoud wants to merge 16 commits into
mainfrom
add-coveralls

Conversation

@Zain-Mahmoud

@Zain-Mahmoud Zain-Mahmoud commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR aims to add coveralls.io support to report test code coverage. Coveralls works by reading a coverage report produced after running the tests and uploads it to coveralls.io. This is our repo link on coveralls: https://coveralls.io/github/cssu/terrain-diffusion. You can see the changes for this PR under pull #52

image image

Validation

  • scripts/quality-check.sh passes locally
  • Appropriate /test commands were run and are passing (e.g. /test gpu)
  • Unit tests were added / e2e tests were added where applicable
  • Manual testing, if applicable (describe further):

Checklist

  • The change is focused and easy to review.
  • Tests were added or updated for behaviour changes.
  • No secrets, generated files, or files that only make sense on my machine are included.
  • Relevant documentation is updated.

Additional Comments

@Zain-Mahmoud Zain-Mahmoud changed the title Add coveralls.io support Add basic coveralls.io support Aug 19, 2026
@Zain-Mahmoud

Copy link
Copy Markdown
Contributor Author

/test all

@github-actions

Copy link
Copy Markdown

Tests passed

  • Groups run: python, slow, web
  • Commit tested: c59c9d9
  • Full log: Actions run
Group Result
python 41/41 passed
slow 0/0 passed
web 1/1 passed

@Zain-Mahmoud
Zain-Mahmoud requested a review from KurbyDoo August 19, 2026 21:20
@coveralls

coveralls commented Aug 19, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 32306386273

Warning

No base build found for commit 295f55d on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 96.765%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 340
Covered Lines: 329
Line Coverage: 96.76%
Coverage Strength: 0.97 hits per line

💛 - Coveralls

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