Skip to content
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

Enable scoverage in github actions and add coveralls plus badge #321

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mdedetrich
Copy link
Contributor

@mdedetrich mdedetrich commented Jul 15, 2021

This PR does the following things

  1. Enables scoverage in github actions so it actually runs on pull requests
  2. Adds coveralls (which works in conjunction with scoverage)
  3. Adds a badge which shows the status of the coverage of the project. Note that it takes roughly a day for coveralls to start displaying the coverage for a new project

ThisBuild / githubWorkflowArtifactUpload := false
ThisBuild / githubWorkflowPublishTargetBranches := Seq()
// This is causing problems with env variables being passed in, see
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure why you specifically disable sbt thin client but in my experience I had to disable it because it doesn't pass ENV variables (which is needed for coveralls) so I just added a comment here explaining why

@mdedetrich mdedetrich force-pushed the enable-scoverage-add-coveralls branch from 80fadee to 593ce06 Compare July 15, 2021 09:12
@mdedetrich mdedetrich force-pushed the enable-scoverage-add-coveralls branch from 593ce06 to 258eab5 Compare July 15, 2021 09:13
@mdedetrich
Copy link
Contributor Author

mdedetrich commented Jul 15, 2021

Hmm, so it seems that this is not working because scoverage doesn't support Scala3 yet. Theoritically its possible to put conditionals so that scoverage/scoveralls is only run for Scala2 but since this is non urgent I think just waiting for Scala3 support in scoverage is better.

@mdedetrich
Copy link
Contributor Author

Upstream issue at scoverage/scalac-scoverage-plugin#299

@mdedetrich mdedetrich marked this pull request as draft July 15, 2021 09:19
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.

1 participant