PDLPorters / PDLPorters/devops
Tell Devel::Cover to ignore files outside current build directory
@zmughal is already working on this.
Since Dec 10, 2021.
- Dominant language
- Shell
- Stars
- 3
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Description
This issue comes up in the coverage results of Test-PDL:
-pdl-commits:#pdl- [Test-PDL] [build/push]: ∀S ✓ a8ba9e2 | Zaki Mughal [sivoais] | Use shared GitHub Actions from PDLPorters (#6)
https://github.com/ebaudrez/Test-PDL/actions/runs/1557475038
[@sivoais] one odd thing with the coverage above: it thinks that a file from the PDL distribution is part of the coverage
[@sivoais] a file that is located in ~/.cpanm ??
[@sivoais] i saw this locally as well, but didn't bother tracking it down in Devel::Cover
[@sivoais] so even though Test-PDL's coverage is really 100%, it comes out at 76% on Coveralls
Coveralls link is: https://coveralls.io/jobs/91303010.
This is possibly just a matter of adding a -ignore_re option to the cover
command, but will need to be tested.
$ $TEST_RUNNER_PREFIX $MYPERL -S cover -test -relative_only -gcov_chdir -report Coveralls
----------------------------------- ------ ------ ------ ------ ------ ------
File stmt bran cond sub time total
----------------------------------- ------ ------ ------ ------ ------ ------
...PDL-2.063/Basic/Core/Types.pm.PL 34.1 19.2 20.0 33.3 43.0 31.0
blib/lib/Test/Deep/PDL.pm 100.0 75.0 n/a 100.0 5.3 97.7
blib/lib/Test/PDL.pm 100.0 100.0 100.0 100.0 51.6 100.0
Total 73.3 69.4 81.8 58.2 100.0 70.3
----------------------------------- ------ ------ ------ ------ ------ ------
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.