Fix warnings

Author: darrelmiller

src/Microsoft.OpenApi.Readers/ParsingContext.cs

@@ -30,8 +30,15 @@ public class ParsingContext
         internal List<OpenApiTag> Tags { get; private set; } = new List<OpenApiTag>();
         internal Uri BaseUrl { get; set; }
 
+        /// <summary>
+        /// Diagnostic object that returns metadata about the parsing process.
+        /// </summary>
         public OpenApiDiagnostic Diagnostic { get; }
 
+        /// <summary>
+        /// Create Parsing Context
+        /// </summary>
+        /// <param name="diagnostic">Provide instance for diagnotic object for collecting and accessing information about the parsing.</param>
         public ParsingContext(OpenApiDiagnostic diagnostic)
         {
             Diagnostic = diagnostic;
@@ -41,7 +48,6 @@ public ParsingContext(OpenApiDiagnostic diagnostic)
         /// Initiates the parsing process.  Not thread safe and should only be called once on a parsing context
         /// </summary>
         /// <param name="yamlDocument">Yaml document to parse.</param>
-        /// <param name="diagnostic">Diagnostic object which will return diagnostic results of the operation.</param>
         /// <returns>An OpenApiDocument populated based on the passed yamlDocument </returns>
         internal OpenApiDocument Parse(YamlDocument yamlDocument)
         {
@@ -77,7 +83,6 @@ internal OpenApiDocument Parse(YamlDocument yamlDocument)
         /// </summary>
         /// <param name="yamlDocument"></param>
         /// <param name="version">OpenAPI version of the fragment</param>
-        /// <param name="diagnostic">Diagnostic object which will return diagnostic results of the operation.</param>
         /// <returns>An OpenApiDocument populated based on the passed yamlDocument </returns>
         internal T ParseFragment<T>(YamlDocument yamlDocument, OpenApiSpecVersion version) where T : IOpenApiElement
         {

src/Microsoft.OpenApi/Any/OpenApiString.cs

@@ -14,6 +14,7 @@ public class OpenApiString : OpenApiPrimitive<string>
         /// Initializes the <see cref="OpenApiString"/> class.
         /// </summary>
         /// <param name="value"></param>
+        /// <param name="isExplicit">Used to indicate if a string is quoted.</param>
         public OpenApiString(string value, bool isExplicit = false)
             : base(value)
         {

src/Microsoft.OpenApi/Extensions/OpenApiSerializableExtensions.cs

@@ -50,6 +50,7 @@ public static void SerializeAsYaml<T>(this T element, Stream stream, OpenApiSpec
         /// <param name="stream">The given stream.</param>
         /// <param name="specVersion">The Open API specification version.</param>
         /// <param name="format">The output format (JSON or YAML).</param>
+        /// <param name="settings">Provide configuration settings for controlling writing output</param>
         public static void Serialize<T>(
             this T element,
             Stream stream,

src/Microsoft.OpenApi/Services/LoopDetector.cs

@@ -13,7 +13,6 @@ internal class LoopDetector
         /// <summary>
         /// Maintain history of traversals to avoid stack overflows from cycles
         /// </summary>
-        /// <param name="loopId">Any unique identifier for a stack.</param>
         /// <param name="key">Identifier used for current context.</param>
         /// <returns>If method returns false a loop was detected and the key is not added.</returns>
         public bool PushLoop<T>(T key)
@@ -39,7 +38,6 @@ public bool PushLoop<T>(T key)
         /// <summary>
         /// Exit from the context in cycle detection
         /// </summary>
-        /// <param name="loopid">Identifier of loop</param>
         public void PopLoop<T>()
         {
             if (_loopStacks[typeof(T)].Count > 0)
@@ -65,7 +63,6 @@ public void SaveLoop<T>(T loop)
         /// <summary>
         /// Reset loop tracking stack
         /// </summary>
-        /// <param name="loopid">Identifier of loop to clear</param>
         internal void ClearLoop<T>()
         {
             _loopStacks[typeof(T)].Clear();

src/Microsoft.OpenApi/Services/OpenApiDifferenceOperation.cs

@@ -3,6 +3,8 @@
 
 namespace Microsoft.OpenApi.Services
 {
+#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
+
     /// <summary>
     /// The open api difference operation.
     /// </summary>
@@ -12,4 +14,6 @@ public enum OpenApiDifferenceOperation
         Remove,
         Update
     }
+#pragma warning restore CS1591 // Missing XML comment for publicly visible type or member
+
 }

src/Microsoft.OpenApi/Writers/OpenApiJsonWriter.cs

@@ -14,6 +14,7 @@ public class OpenApiJsonWriter : OpenApiWriterBase
         /// Initializes a new instance of the <see cref="OpenApiJsonWriter"/> class.
         /// </summary>
         /// <param name="textWriter">The text writer.</param>
+        /// <param name="settings">Settings for controlling how the OpenAPI document will be written out.</param>
         public OpenApiJsonWriter(TextWriter textWriter, OpenApiWriterSettings settings = null) : base(textWriter, settings)
         {
         }

test/Microsoft.OpenApi.SmokeTests/ApiGurus.cs

@@ -71,7 +71,7 @@ JToken GetProp(JToken obj, string prop)
         }
 
         // Disable as some APIs are currently invalid [Theory(DisplayName = "APIs.guru")]
-        [MemberData(nameof(GetSchemas))]
+        // [MemberData(nameof(GetSchemas))]
         public async Task EnsureThatICouldParse(string url)
         {
             var response = await _httpClient.GetAsync(url);