Replies: 1 comment
-
Hi, you can go ahead to create PR then I can merge it |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
An aesthetic, cosmetic issue, for sure, but nonetheless, here it goes:
This:
Is not idiomatic C#. In this particular case, since you're using the dotnet configuration API you might simply map the config section to your configuration object:
Also here:
it should be using object initializer syntax:
Same with many other code samples.
I can create a PR fixing this if you want.
Beta Was this translation helpful? Give feedback.
All reactions