Skip to content
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

Add docs for new console logging APIs #4918

Closed
wants to merge 1 commit into from

Conversation

maryamariyan
Copy link
Member

@maryamariyan maryamariyan commented Sep 30, 2020

Summary

Adds docs for new console logging APIs.

cc: @carlossanlop

@maryamariyan
Copy link
Member Author

cc: @safern

Copy link
Member

@safern safern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc: @gewarren

@opbld34
Copy link

opbld34 commented Sep 30, 2020

Docs Build status updates of commit 5bbad4f:

⚠️ Validation status: warnings

File Status Preview URL Details
xml/Microsoft.Extensions.DependencyInjection/PollyServiceCollectionExtensions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Logging.Console/ConsoleFormatter.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/ConsoleFormatterNames.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/ConsoleFormatterOptions.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/JsonConsoleFormatterOptions.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/LoggerColorBehavior.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/SimpleConsoleFormatterOptions.xml ✅Succeeded View

xml/Microsoft.Extensions.DependencyInjection/PollyServiceCollectionExtensions.xml

  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.PolicyRegistry'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IReadOnlyPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IReadOnlyPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.PolicyRegistry'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.

For more details, please refer to the build report.

If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@opbld32
Copy link

opbld32 commented Sep 30, 2020

Docs Build status updates of commit c26a06f:

⚠️ Validation status: warnings

File Status Preview URL Details
xml/Microsoft.Extensions.DependencyInjection/PollyServiceCollectionExtensions.xml ⚠️Warning View Details
xml/Microsoft.Extensions.Logging.Console/ConsoleFormatter.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/ConsoleFormatterNames.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/ConsoleFormatterOptions.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/JsonConsoleFormatterOptions.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/LoggerColorBehavior.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/SimpleConsoleFormatterOptions.xml ✅Succeeded View

xml/Microsoft.Extensions.DependencyInjection/PollyServiceCollectionExtensions.xml

  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.PolicyRegistry'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IReadOnlyPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IReadOnlyPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.PolicyRegistry'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.
  • Line 0, Column 0: [Warning-xref-not-found] Cross reference not found: 'Polly.Registry.IPolicyRegistry`1'.

For more details, please refer to the build report.

If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@maryamariyan
Copy link
Member Author

closing in favor of #4925 (review)

<param name="scopeProvider">The provider of scope data.</param>
<param name="textWriter">The string writer embedding ansi code for colors.</param>
<summary>Writes the log message to the specified TextWriter.</summary>
<remarks>if the formatter wants to write colors to the console, it can do so by embedding ANSI color codes into the string.</remarks>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<remarks>if the formatter wants to write colors to the console, it can do so by embedding ANSI color codes into the string.</remarks>
<remarks>If the formatter wants to write colors to the console, it can do so by embedding ANSI color codes into the string.</remarks>

<param name="logEntry">The log entry.</param>
<param name="scopeProvider">The provider of scope data.</param>
<param name="textWriter">The string writer embedding ansi code for colors.</param>
<summary>Writes the log message to the specified TextWriter.</summary>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<summary>Writes the log message to the specified TextWriter.</summary>
<summary>Writes the log message to the specified text writer.</summary>

@@ -34,7 +34,7 @@
<ReturnType>System.String</ReturnType>
</ReturnValue>
<Docs>
<summary>To be added.</summary>
<summary>Reserved name for json console formatter.</summary>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<summary>Reserved name for json console formatter.</summary>
<summary>The name reserved for the JSON console formatter.</summary>

@@ -74,7 +74,7 @@
<ReturnType>System.String</ReturnType>
</ReturnValue>
<Docs>
<summary>To be added.</summary>
<summary>Reserved name for systemd console formatter</summary>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<summary>Reserved name for systemd console formatter</summary>
<summary>Reserved name for the systemd console formatter.</summary>

@@ -14,7 +14,7 @@
</Base>
<Interfaces />
<Docs>
<summary>To be added.</summary>
<summary>Reserved formatter names for the built-in console formatters.</summary>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<summary>Reserved formatter names for the built-in console formatters.</summary>
<summary>Provides reserved formatter names for the built-in console formatters.</summary>

@@ -74,7 +75,7 @@
</ReturnValue>
<MemberValue>1</MemberValue>
<Docs>
<summary>To be added.</summary>
<summary>Enable color for logging.</summary>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<summary>Enable color for logging.</summary>
<summary>Use color for logging.</summary>

@@ -14,7 +14,7 @@
</Base>
<Interfaces />
<Docs>
<summary>To be added.</summary>
<summary>Options for the built-in default console log formatter.</summary>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<summary>Options for the built-in default console log formatter.</summary>
<summary>Specifies options for the built-in, default console log formatter.</summary>

@@ -51,7 +51,7 @@
<ReturnType>Microsoft.Extensions.Logging.Console.LoggerColorBehavior</ReturnType>
</ReturnValue>
<Docs>
<summary>To be added.</summary>
<summary>Determines when to use color when logging messages.</summary>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<summary>Determines when to use color when logging messages.</summary>
<summary>Gets or sets a value that indicates when to use color for log messages.</summary>

@@ -72,7 +72,7 @@
<ReturnType>System.Boolean</ReturnType>
</ReturnValue>
<Docs>
<summary>To be added.</summary>
<summary>When <see langword="true" />, the entire message gets logged in a single line.</summary>
<value>To be added.</value>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<value>To be added.</value>
<value><see langword="true" />to log the entire message as a single line; otherwise, <see langword=false />.</value>

@@ -72,7 +72,7 @@
<ReturnType>System.Boolean</ReturnType>
</ReturnValue>
<Docs>
<summary>To be added.</summary>
<summary>When <see langword="true" />, the entire message gets logged in a single line.</summary>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<summary>When <see langword="true" />, the entire message gets logged in a single line.</summary>
<summary>Gets or sets a value that indicates whether the entire message should be logged as a single line.</summary>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants