MishaKav / jest-coverage-comment

Comments a pull request or commit with the jest code coverage badge, full report and tests summary
MIT License
80 stars 34 forks source link

Report only changed files not working #88

Closed SirPhemmiey closed 5 months ago

SirPhemmiey commented 5 months ago

Hello again,

  1. I get this error each time i set report-only-changed-files: true in my configuration
Screenshot 2024-05-16 at 13 54 37

Do you know how i can fix this?

  1. Clicking on the files and lines here doesn't link to the corresponding location. This might be a long stretch and it's not a make or break for me but do you know how i can possibly get it to work correctly? Screenshot 2024-05-16 at 12 45 49
MishaKav commented 5 months ago
  1. Resource not accessible by integration - it's a known issue, you may try to find suggestions on how to fix it https://github.com/MishaKav/pytest-coverage-comment/issues/68
  2. I need to see the full generated link to help you. Usually, it's one of the problems:

P.S. I know that the links that I suggest are not the same repo, but both of them have similar issues and almost same api.

SirPhemmiey commented 5 months ago

Thank you for the references.

On #2, i'm running in a monorepo and the structure is like apps//src/**

If i passcoverage-path-prefix: apps/, it will go to somewhere like apps/src/components/renderer.element.ts instead of apps/<app-name>/src/components/renderer.element.ts

this is probably out of scope of your library, but i'm happy to hear your thoughts

MishaKav commented 5 months ago

I think you may run this github-action several times, and each time generate a report for a different app with a specific prefix (you will get a different comment for each app - which is a good approach I think), because even if you use multiple-files, your report will be generated for one app.