Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

c# support #32

Open
noname77 opened this issue Oct 31, 2023 · 3 comments
Open

c# support #32

noname77 opened this issue Oct 31, 2023 · 3 comments

Comments

@noname77
Copy link

noname77 commented Oct 31, 2023

Hi,

This project looks great, thanks!

I tried it with doxygen output for a c# project and run into some issues where the coverage is not reflected correctly.

I was wondering if this is meant to be supported and if you could point me in the right direction?

Unfortunately I cannot share the project and didn't get around to creating a minimum reproducible example, but the two issues im seeing at first glance are:

  • top level file comment block is incorrectly recognised as code giving a false miss

EDIT: this is same as #15

Screenshot 2023-10-31 at 13 50 24

  • c# properties are not recognised as code, but methods work as expected

Screenshot 2023-10-31 at 13 51 54

I'd be happy to share more debug output, create a minimum example or test stuff or submit a PR if this project scope would include c# support.

Best,
wiktor

@noname77
Copy link
Author

I decided to give it a go and the property issue turned out to be a simple fix, just added the property kind, see noname77@3aa4065

$ python3 -m coverxygen --xml-dir doxygen/doxyxml --src-dir ../ --output doc-coverage.info --kind enum,enumvalue,variable,function,class,define,namespace,property

I have a feeling there will be more missing kinds, so not submitting a PR just yet, changes will be tracked in master...noname77:coverxygen:feat/csharp-support

@noname77
Copy link
Author

noname77 commented Oct 31, 2023

To leave a trace somewhere, I added the interface kind in noname77@8f949dd and it works for a simple definition

Screenshot 2023-10-31 at 15 05 28

but for some reason it fails on a generic interface:

image

weirdly, a similar class is hadndled correctly

Screenshot 2023-10-31 at 15 09 59

@xmarcelet
Copy link

I'll be happy to merge your PR if can provide one 👍

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

No branches or pull requests

2 participants