Add support for primary constructors in LoggerMessageGenerator

dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.

https://docs.microsoft.com/dotnet/core/

MIT License

14.23k stars 4.45k forks source link

Add support for primary constructors in LoggerMessageGenerator #101660

Open kimsey0 opened 2 weeks ago

kimsey0 commented 2 weeks ago

Fixes #91121.

tarekgh commented 2 weeks ago

@kimsey0 thanks for submitting the PR. I converted it to draft till it is ready for actual review.

kimsey0 commented 2 weeks ago

@tarekgh, with the helpful suggestion from Cyrus, I now think this is ready for review. Should I mark it as such or wait for more feedback, perhaps from @eiriktsarpalis? (I don't know when he's back from vacation.)