SteveDunn / Vogen

A semi-opinionated library which is a source generator and a code analyser. It Source generates Value Objects
Apache License 2.0
887 stars 46 forks source link

[Cosmetic] 4.0.3 generates a warning in XML documentation #588

Closed CheloXL closed 6 months ago

CheloXL commented 6 months ago

Describe the bug

Trying to compile a project where I have set warnings as errors + generate documentation on build. It seems that the newly added method TryFrom has some errors in the XML documentation, as I'm getting the following:

0>DDS.Domain_ETag.g.cs(122,12): Warning CS1571 : XML comment has a duplicate param tag for 'value'
0>DDS.Domain_ETag.g.cs(128,11): Warning CS1573 : Parameter 'vo' has no matching param tag in the XML comment for 'ETag.TryFrom(string, out ETag)' (but other parameters do)

Peeking into the generated code I see where the error comes from: image I believe the second param name should be vo.

Steps to reproduce

Try to compile a project that has at least one VO, with the above settings set to true.

Expected behaviour

No errors at compile time.

Reonekot commented 6 months ago

This hit me as well and it's borderline "Cosmetic" IMO - as you said, upgrading and having warnings as errors broke the build and took me some time to figure out the root cause. But Cosmetic in the sense that you can ignore ignore the warning in the project.