Skip to content

Conversation

@tm1000
Copy link
Member

@tm1000 tm1000 commented May 16, 2024

fixes #52

I'm not sure what to do if sources has more than one string but I suppose that can be for the next iteration of this (or when someone sends me one with multiples)

@github-actions
Copy link

Code Coverage Report

Code Coverage

Package Base Coverage New Coverage Difference
main.ts 🔴 0% 🔴 0% ⚪ 0%
reports/clover/index.ts 🟠 66.67% 🟢 100% 🟢 33.33%
reports/clover/parser/index.ts 🔴 3.92% 🟢 84.31% 🟢 80.39%
reports/clover/types/index.ts 🟢 100% 🟢 100% ⚪ 0%
reports/cobertura/index.ts 🟠 66.67% 🟢 100% 🟢 33.33%
reports/cobertura/parser/index.ts 🔴 13.33% 🟢 92.31% 🟢 78.98%
reports/cobertura/types/index.ts 🟢 100% 🟢 100% ⚪ 0%
utils.ts 🟠 59.4% 🟠 72.22% 🟢 12.82%
Overall Coverage 🟢 36.46% 🟢 54.58% 🟢 18.12%

Minimum allowed coverage is 0%, this run produced 54.58%

@hsingh-texada
Copy link

hsingh-texada commented May 16, 2024

@tm1000 Would you also be able to add a flag, so that only classes for which coverage changed gets added to the MD file.. If a project has thousands of classes, it would end up being a really long file and difficult to read in a PR context.

@tm1000
Copy link
Member Author

tm1000 commented May 16, 2024

@hsingh-texada I added #58 which I might get to tomorrow evening if all goes well.

@kuhnroyal
Copy link
Contributor

Awesome work! @hsingh-texada Also really love this suggestion!

@tm1000 tm1000 merged commit f5234ea into main May 16, 2024
@tm1000 tm1000 deleted the tm1000/issue52 branch May 16, 2024 20:34
@kuhnroyal
Copy link
Contributor

I am getting an error.

Error: Cannot read properties of undefined (reading 'map')

See https://github.com/cfug/dio/actions/runs/9137019517/job/25139180215

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support cobertura class level coverage

6 participants