Skip to content

See what happens#307

Closed
vkostyukov wants to merge 1 commit intodevelopfrom
vk/sbt-leave-us-alone
Closed

See what happens#307
vkostyukov wants to merge 1 commit intodevelopfrom
vk/sbt-leave-us-alone

Conversation

@vkostyukov
Copy link
Copy Markdown
Contributor

No description provided.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 27, 2022

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.30%. Comparing base (936ceec) to head (34b8db8).
⚠️ Report is 103 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #307      +/-   ##
===========================================
+ Coverage    49.11%   49.30%   +0.18%     
===========================================
  Files          320      322       +2     
  Lines        18459    18532      +73     
  Branches      1074     1062      -12     
===========================================
+ Hits          9067     9138      +71     
- Misses        9392     9394       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@felixbr
Copy link
Copy Markdown
Contributor

felixbr commented Jul 28, 2022

Hi,

I noticed you're hitting similar trouble with sbt-scoverage as I have recently. Maybe it would be helpful to run coverage only for the main Scala version (i.e. 2.13.6 currently) instead of all versions.

The code and tests are the same either way, so imo it would be sufficient to only do it for the main version and it might make the sbt-scoverage version stuff easier.

You can take a look at #308 as an example where I've excluded coverage for Scala 3 and it seems to work nicely.

Cheers
~ Felix

edit: There is also a coverageEnabled setting in sbt, which might be helpful but I didn't have too much luck with it, so I added conditionals in the CI-build instead.

@bryce-anderson
Copy link
Copy Markdown
Contributor

Looks like it worked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants