Closed bigtlb closed 3 years ago
Thanks for the contribution! I agree that hiding IEquatable<T>
and the synthesized members makes sense, except for the public properties.
Please advise on how I can make the build succeed.
Looks like XML documentation comments didn't work correctly with records under 5.0.100. I've changed GitHub Actions to use the latest .NET 5 SDK; perhaps merging that in that will fix the PR build.
You should also run .\build.ps1 generate-docs
and commit those changes.
Thanks. I was just noticing that part.
Should be good now. it build on my branch.
Rebased and amended: #121. Thanks again!
C# 9
record
type should work similar toclass
but it has some built in members and a specific inheritance, which may be better off hidden.IsRecord
AndTypeKind.Record
updated generation code to outputrecord
in place ofclass
, but also consider record inn most places where classes are considered (although it can't inherit).elses
can be
Suggested next steps is hide
IEquatable<T>
inheritance and builtin members.