-
Notifications
You must be signed in to change notification settings - Fork 61
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
[infra] Coverage issues #812
Comments
The source not available problem might be because you have not signed in? |
Thanks! I wonder why one needs to be signed in to see the source code of a public repo? 🤔 |
I had wondered the same before! Btw the PR doesn't seem to work, I also pushed some code to trigger jnigen's CI as well, but coverall still only shows ffigen's coverage. |
The flag name seems to be ignored for ffigen: https://coveralls.io/builds/64057741 The only difference I can see is |
It looks like coverage is now working as expected: #819 (comment) |
Now that we have added coverage from multiple workflows, Coveralls is forgetting the coverage from other workflows.

FFIgen CI ran last:

This is a reintroduction of #10:
Secondly, Coveralls doesn't seem to find the source file locations:

We should either see if we can fix the setup for Coveralls, or switch to CodeCov (#70 (comment)).
cc @HosseinYousefi @liamappelbe
The text was updated successfully, but these errors were encountered: