-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Microsoft.Extensions.Logging.TraceSource package readme update #92962
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
Microsoft.Extensions.Logging.TraceSource package readme update #92962
Conversation
List related packages with adding short description and direct URI to nuget.org package. Fix dotnet#92228
List API Documentation link in Additional Documentation section. Fix dotnet#92228
Liste main types provided by library in Main Types section. Does not include internal types which are not exposed out. Fix dotnet#92228
List key features of the library. Add a short package description. Fix dotnet#92228
|
Tagging subscribers to this area: @dotnet/area-extensions-logging Issue DetailsThis makes progress towards #92228 I am eager to hear your thoughts! If any changes required, let me know.
|
tarekgh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @illialarka
| <!-- The related packages associated with this package --> | ||
|
|
||
| * Abstractions for dependency injection: [Microsoft.Extensions.DependencyInjection.Abstractions](https://www.nuget.org/packages/Microsoft.Extensions.DependencyInjection.Abstractions/) | ||
| * Defult implementation of logging infrastructure: [Microsoft.Extensions.Logging](https://www.nuget.org/packages/Microsoft.Extensions.Logging/) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo: Default.
thanks for helping supply content!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! Thank you! I have created one more PR with fix for this typo right here. #92989
Thank you @danmoseley
This makes progress towards #92228
I am eager to hear your thoughts! If any changes required, let me know.