Added doc comments for main APIs:

- Coherence-Signed#75

Author: DamianEdwards

src/Microsoft.Extensions.Logging.Abstractions/ILogger.cs

@@ -6,7 +6,7 @@
 namespace Microsoft.Extensions.Logging
 {
     /// <summary>
-    /// A generic interface for logging.
+    /// Represents a type used to perform logging.
     /// </summary>
     public interface ILogger
     {

src/Microsoft.Extensions.Logging.Abstractions/ILoggerFactory.cs

@@ -6,22 +6,27 @@
 namespace Microsoft.Extensions.Logging
 {
     /// <summary>
-    /// Used to create logger instances of the given name.
+    /// Represents a type used to configure the logging system and create instances of <see cref="ILogger"/> from
+    /// the registered <see cref="ILoggerProvider"/>s.
     /// </summary>
     public interface ILoggerFactory : IDisposable
     {
         /// <summary>
-        /// The minimum level of log messages sent to registered loggers.
+        /// The minimum level of log messages sent to loggers.
         /// </summary>
         LogLevel MinimumLevel { get; set; }
 
         /// <summary>
-        /// Creates a new ILogger instance of the given name.
+        /// Creates a new <see cref="ILogger"/> instance.
         /// </summary>
-        /// <param name="categoryName"></param>
-        /// <returns></returns>
+        /// <param name="categoryName">The category name for messages produced by the logger.</param>
+        /// <returns>The <see cref="ILogger"/>.</returns>
         ILogger CreateLogger(string categoryName);
 
+        /// <summary>
+        /// Adds an <see cref="ILoggerProvider"/> to the logging system.
+        /// </summary>
+        /// <param name="provider">The <see cref="ILoggerProvider"/>.</param>
         void AddProvider(ILoggerProvider provider);
     }
 }

src/Microsoft.Extensions.Logging.Abstractions/ILoggerOfT.cs

@@ -6,9 +6,11 @@
 namespace Microsoft.Extensions.Logging
 {
     /// <summary>
-    /// A generic interface for logging where the category name is taken from the specified TCategoryName.
-    /// For enabling activation of named ILogger from DI.
+    /// A generic interface for logging where the category name is derived from the specified
+    /// <paramref name="TCategoryName"/> type name.
+    /// Generally used to enable activation of a named <see cref="ILogger"/> from dependency injection.
     /// </summary>
+    /// <typeparam name="TCategoryName">The type who's name is used for the logger category name.</typeparam>
     public interface ILogger<out TCategoryName> : ILogger
     {
 

src/Microsoft.Extensions.Logging.Abstractions/ILoggerProvider.cs

@@ -6,15 +6,15 @@
 namespace Microsoft.Extensions.Logging
 {
     /// <summary>
-    /// Used to create logger instances of the given name.
+    /// Represents a type that can create instances of <see cref="ILogger"/>.
     /// </summary>
     public interface ILoggerProvider : IDisposable
     {
         /// <summary>
-        /// Creates a new ILogger instance of the given name.
+        /// Creates a new <see cref="ILogger"/> instance.
         /// </summary>
-        /// <param name="name"></param>
+        /// <param name="categoryName">The category name for messages produced by the logger.</param>
         /// <returns></returns>
-        ILogger CreateLogger(string name);
+        ILogger CreateLogger(string categoryName);
     }
 }

src/Microsoft.Extensions.Logging.Abstractions/LogLevel.cs

@@ -3,13 +3,44 @@
 
 namespace Microsoft.Extensions.Logging
 {
+    /// <summary>
+    /// Defines logging severity levels.
+    /// </summary>
     public enum LogLevel
     {
+        /// <summary>
+        /// Logs that contain the most detailed messages. These messages may contain sensitive application data.
+        /// These messages are disabled by default and should never be enabled in a production environment.
+        /// </summary>
         Debug = 1,
+
+        /// <summary>
+        /// Logs that are used for interactive investigation during development.  These logs should primarily contain
+        /// information useful for debugging and have no long-term value.
+        /// </summary>
         Verbose = 2,
+
+        /// <summary>
+        /// Logs that track the general flow of the application. These logs should have long-term value.
+        /// </summary>
         Information = 3,
+
+        /// <summary>
+        /// Logs that highlight an abnormal or unexpected event in the application flow, but do not otherwise cause the
+        /// application execution to stop.
+        /// </summary>
         Warning = 4,
+
+        /// <summary>
+        /// Logs that highlight when the current flow of execution is stopped due to a failure. These should indicate a
+        /// failure in the current activity, not an application-wide failure.
+        /// </summary>
         Error = 5,
+
+        /// <summary>
+        /// Logs that describe an unrecoverable application or system crash, or a catastrophic failure that requires
+        /// immediate attention.
+        /// </summary>
         Critical = 6,
     }
 }