Skip to content

Conversation

43081j
Copy link
Contributor

@43081j 43081j commented Jun 24, 2025

Switches to using vitest's v8 coverage provider which should be faster
and more accurate.

Depends on #4806 and #4808

Copy link

github-actions bot commented Jun 24, 2025

📊 Tachometer Benchmark Results

Summary

A summary of the benchmark results will show here once they finish.

Results

The full results of your benchmarks will show here once they finish.

tachometer-reporter-action v2 for CI

@JoviDeCroock
Copy link
Member

JoviDeCroock commented Jun 25, 2025

Thinking more about this, how does this work with coveralls?

- name: Coveralls GitHub Action

I can see

🔍 Detected coverage file: coverage/lcov.info
🚨 Nothing to report

@43081j
Copy link
Contributor Author

43081j commented Jun 25, 2025

That's the same output before this change. If you look at another branch that doesn't include this, it is the same

I think it just means there's no coverage errors, i.e thresholds were met

When I'm at a laptop again I can have a look at the report it outputs to be sure it is right still

Switches to using vitest's v8 coverage provider which should be faster
and more accurate.
@43081j
Copy link
Contributor Author

43081j commented Jun 25, 2025

@JoviDeCroock you were right, it is fixed now 👍

@JoviDeCroock JoviDeCroock merged commit 021d60d into preactjs:main Jun 25, 2025
5 checks passed
@43081j 43081j deleted the vitest-v8 branch June 25, 2025 10:39
@JoviDeCroock JoviDeCroock mentioned this pull request Aug 19, 2025
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