search

romeovs / lcov-reporter-action

Comments a pull request with the code coverage generated by your tests.
MIT License
161 stars 99 forks source link

Links in coverage report don't work #45

Open mieszkoWrzeszczynskiMtab opened 2 years ago

mieszkoWrzeszczynskiMtab commented 2 years ago

In the coverage report, there are links to see files or uncovered lines of code. Unfortunately, these links don't work (there is 404). Even in your README, it seems to not work as well.

harry-comparis commented 1 year ago

It actually works, it depends from which location/directory you create the coverage from. It will use the paths provided in your *.lcov file and add them to the html report. The paths provided in the *.lcov file have to be aligned with your repository structure.

hramrach commented 9 months ago

Which they aren't if you build both the base branch and the current branch because there is no way to store and access the base branch data, and you cannot build base as subdirectory of current because that's a mess.

PupilTong commented 2 months ago

@hramrach How about add a new configuration lcovBasePath to add a base path on all links?