-
Notifications
You must be signed in to change notification settings - Fork 410
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
trivial: ci: install coverage #7187
base: main
Are you sure you want to change the base?
Conversation
hopefully fixes coveralls
Well so that's interesting - it fails on 0.6.9 too! I wonder if this is a fwupd problem. |
@superm1 what does coverage.xml show? Is it an empty file or something weird perhaps? |
let's see, I just changed the job to save it to artifacts so we can tell. |
It's empty! |
My conclusion from that finding is something must have changed in the Debian container that broke coverage. It just starts to show up today because containers regenerated about 16 hours ago. |
Can we compare the .debs that were used in each? |
It's a bit difficult to run diff on without some fancy pants filtering tools. You got any good ways? old_2_push_to_registry (debian-x86_64).txt |
But at least for the biggies.
|
hopefully fixes coveralls
Type of pull request: