NotificationsYou must be signed in to change notification settings
Fork1.1k
Star6.2k

Make coverage support incremental compilation#24773

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

lolgab wants to merge2 commits intoscala:main

base:main

Choose a base branch

fromlolgab:incremental-coverage

Open

Make coverage support incremental compilation#24773

lolgab wants to merge2 commits intoscala:mainfromlolgab:incremental-coverage

+163 −11

Conversation

Copy link

Contributor

lolgab commentedDec 18, 2025

Port ofscoverage/scalac-scoverage-plugin#742 for Scala 3.

tgodzik self-assigned this

Dec 18, 2025

Copy link

Contributor

tgodzik commentedDec 18, 2025

Thanks for working on this. Let me know when this is ready! Seems like github actions are having a tough day though, so we'll need to rerun some jobs.

Copy link

ContributorAuthor

lolgab commentedDec 18, 2025

Thanks for working on this. Let me know when this is ready! Seems like github actions are having a tough day though, so we'll need to rerun some jobs.

I want to port the tests as well, but I'm having trouble running the tests.

I get:

sbt:scala3-nonbootstrapped> last scala3-library-nonbootstrapped / scalaInstance[error] java.lang.RuntimeException: Missing Scala tool configuration from the 'update' report.  Explicitly define scalaInstance or scalaHome or include Scala dependencies in the 'scala-tool' configuration.[error]         at scala.sys.package$.error(package.scala:30)[error]         at sbt.Defaults$.$anonfun$scalaInstanceFromUpdate$10(Defaults.scala:1214)[error]         at scala.Option.getOrElse(Option.scala:189)[error]         at sbt.Defaults$.$anonfun$scalaInstanceFromUpdate$1(Defaults.scala:1214)[error]         at scala.Function1.$anonfun$compose$1(Function1.scala:49)[error]         at......[error] (scala3-library-nonbootstrapped / scalaInstance) Missing Scala tool configuration from the 'update' report.  Explicitly define scalaInstance or scalaHome or include Scala dependencies in the 'scala-tool' configuration.

Copy link

Contributor

tgodzik commentedDec 18, 2025

I think for coverage it should besbt scala3-compiler-nonbootstrapped/testOnly dotty.tools.dotc.coverage.CoverageTests does it work for you?

Copy link

ContributorAuthor

lolgab commentedDec 18, 2025

@tgodzik Thank you. I needed to cleanup my workspace directory since I had last compiled Scala3 a long time ago and something was broken in thetarget directories. I can run tests now