Add code coverage check in each pr

[muraca]

fix #86

[pepoviola]

Hi @muraca, Thanks for working on this 👍. I think something like this will work.

    - name: Setup LCOV
      uses: hrishikesh-kadam/setup-lcov@v1
    - name: Report code coverage
      uses: zgosalvez/github-actions-report-lcov@v3
      with:
        coverage-files: lcov.*.info
        artifact-name: code-coverage-report
        github-token: ${{ secrets.GITHUB_TOKEN }}
        update-comment: true

Thanks!

[muraca]

@pepoviola you noticed too fast, sorry if I'm making a mess but I noticed too late that I opened a PR on this repo instead of my personal one.

I didn't want to add more bloat by installing lcov too, it is used to create an html file we don't need here, and that's why I moved from zgosalvez/github-actions-report-lcov@v3 to Nef10/lcov-reporter-action@v0.4.0.

Now I get a 404 error because the action makes a GET request to $CORRECT_URL/ and it should be without the /.

[pepoviola]

Hi @muraca, I fixed the request (the issue was related on how to get the pr number). But now is throwing a diff error because can't create a comment with the info.

status: 403,
  response: {
    url: 'https://api.github.com/repos/paritytech/zombienet-sdk/issues/88/comments',
    status: 403,

[muraca]

Looks like some permissions are missing.
Anyway I will move the coverage part to a new job, which runs only after build jobs are completed and only if it is a pull request.

[muraca]

CI run for dummy pull request.

Comment works:

[pepoviola]

Hi @muraca, great! Yes, I wonder if the permission issue is related to be a fork. I would merge this one and we can check with the prs in this repo. Thanks again!!

[pepoviola]

Hi @muraca, just to followup this works with prs from the repo (#89). Thanks!!!

[muraca]

Happy to help!

[l0r1s]

Thanks @muraca ! 🚀