middleware icon indicating copy to clipboard operation
middleware copied to clipboard

Codecov CI issues

Open yusukebe opened this issue 5 months ago • 6 comments

Codecov running on CI needs improvement. This result shows it.

https://github.com/honojs/middleware/pull/1360/checks

1. codecov/project fails

Image

2. All packages are covered

Ideally, only related packages should be covered.

Image

yusukebe avatar Aug 10 '25 06:08 yusukebe

Hi @BarryThePenguin

Can you investigate it?

yusukebe avatar Aug 10 '25 06:08 yusukebe

Sure thing 👍🏻

I've been waiting for vitest v4 to be released which should help somewhat.. #1265

It's now in beta, so maybe we can merge that?

BarryThePenguin avatar Aug 10 '25 06:08 BarryThePenguin

Ah, I see. It's a Vitest side issue! So, only v4 can fix it? Or we can still use v3 and fix it?

yusukebe avatar Aug 10 '25 08:08 yusukebe

Yes, it's something to do with how Vitest generates coverage reports for each project. #1265 is passing using v4.0.0-beta.2, but failing with newer beta releases

honojs/middleware is regularly tested against main branch of Vitest. You can see what's failing https://github.com/vitest-dev/vitest-ecosystem-ci/actions/runs/16928252703/job/47968182844

It looks to be specifically related to @cloudflare/vitest-pool-workers

BarryThePenguin avatar Aug 14 '25 10:08 BarryThePenguin

Ah, vitest-dev/vitest-ecosystem-ci! It's great that this repo is in it. So, is it a good idea to wait for the official release of v4?

yusukebe avatar Aug 15 '25 00:08 yusukebe

Yes, I think it's a good idea to wait. Jumping in the beta will just create more maintenance headaches 🫨

BarryThePenguin avatar Aug 15 '25 00:08 BarryThePenguin