Skip to content
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

Setup ci for gcov test #208

Open
nuomi2021 opened this issue Apr 4, 2024 · 3 comments
Open

Setup ci for gcov test #208

nuomi2021 opened this issue Apr 4, 2024 · 3 comments
Assignees
Milestone

Comments

@nuomi2021
Copy link
Member

Maybe we can use https://github.com/marketplace/actions/gcovr-action to generate code coverage reports for PR.

got something like http://coverage.ffmpeg.org/index.vvc_intra.c.5d0b519a39871515a1754ee8847b6d69.html#l678

@nuomi2021 nuomi2021 added this to the FFmpeg7.1 milestone Apr 4, 2024
@frankplow
Copy link
Collaborator

I have an old repo where I set up gcov on GitHub Actions, so I can look at porting it maybe. Would you want to test the coverage of the FATE tests or the ffvvc/tests suite? And where would the results be hosted?

@nuomi2021
Copy link
Member Author

nuomi2021 commented Apr 4, 2024

The fate test. our target is to make sure fate covers 95% of the code and the most important branch.

And where would the results be hosted?

All action results will be listed here. https://github.com/ffvvc/FFmpeg/actions
You can

  1. push the latest code to ffvvc/up
  2. cherry-pick 1a3c60a
  3. make the change
  4. send pr to ffvvc/up
  5. we can get the result from https://github.com/ffvvc/FFmpeg/actions

@nuomi2021 nuomi2021 added this to ffvvc Apr 5, 2024
@nuomi2021 nuomi2021 moved this to Todo in ffvvc Apr 5, 2024
@nuomi2021 nuomi2021 moved this from Todo to In Progress in ffvvc Apr 5, 2024
@nuomi2021
Copy link
Member Author

@frankplow , have time for this?
thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

No branches or pull requests

2 participants