some suggestions by rpetrusha

Co-Authored-By: Ron Petrusha <[email protected]>

Author: carlossanlop

xml/System.Text.RegularExpressions/GroupCollection.xml

@@ -105,7 +105,7 @@
       <Docs>
         <param name="key">A string with the name of the captured group to locate.</param>
         <summary>Determines whether the group collection contains a captured group identified by the specified name.</summary>
-        <returns><see langword="true"/> if the group collection contains a captured group identified by the specified key; <see langword="false" /> otherwise.</returns>
+        <returns><see langword="true"/> if the group collection contains a captured group identified by <paramref name="key" />; <see langword="false" /> otherwise.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -210,17 +210,17 @@
         <Parameter Name="arrayIndex" Type="System.Int32" Index="1" FrameworkAlternate="netcore-2.0;netcore-2.1;netcore-2.2;netcore-3.0;netstandard-2.1" />
       </Parameters>
       <Docs>
-        <param name="array">The one-dimensional <see cref="T:System.Text.RegularExpressions.Group[]" /> array that is the destination of the elements copied from the group collection. The array must have zero-based indexing.</param>
-        <param name="arrayIndex">The zero-based index in the <see cref="T:System.Text.RegularExpressions.Group[]" /> array at which copying begins.</param>
+        <param name="array">The one-dimensional array that is the destination of the elements copied from the group collection. The array must have zero-based indexing.</param>
+        <param name="arrayIndex">The zero-based index in <paramref name="array" /> at which copying begins.</param>
         <summary>Copies the elements of the group collection to a <see cref="T:System.Text.RegularExpressions.Group[]" /> array, starting at a particular array index.</summary>
         <remarks>To be added.</remarks>
         <exception cref="T:System.ArgumentNullException"><paramref name="array" /> is null.</exception>
         <exception cref="T:System.ArgumentOutOfRangeException"><paramref name="arrayIndex" /> is less than zero.
 
 -or-
 
-<paramref name="arrayIndex" /> is greater than the length of <see cref="P:System.Text.RegularExpressions.Group[]" />.</exception>
-        <exception cref="T:System.ArgumentException">The length of the <paramref name="array" /> - the <paramref name="arrayIndex" /> is less than the group collection count.</exception>
+<paramref name="arrayIndex" /> is greater than the length of <paramref name="array" />.</exception>
+        <exception cref="T:System.ArgumentException">The length of <paramref name="array" /> - <paramref name="arrayIndex" /> is less than the group collection count.</exception>
       </Docs>
     </Member>
     <Member MemberName="Count">
@@ -586,7 +586,7 @@
       </ReturnValue>
       <Docs>
         <summary>Gets a string enumeration that contains the name keys of the group collection.</summary>
-        <value>A string enumeration.</value>
+        <value>The name keys of the group collection.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -671,8 +671,8 @@
         <Parameter Name="item" Type="System.Text.RegularExpressions.Group" Index="0" FrameworkAlternate="netcore-2.0;netcore-2.1;netcore-2.2;netcore-3.0;netstandard-2.1" />
       </Parameters>
       <Docs>
-        <param name="item">This method is not supported.</param>
-        <summary>Implementation from <see cref="T:System.Collections.Generic.ICollection" />. This method is not supported.</summary>
+        <param name="item">The group to add to the collection.</param>
+        <summary>Adds a <see cref="T:System.Text.RegularExpressions.Group" /> to the collection. Calling this method always throws <see cref="T:System.NotSupportedException" />.</summary>
         <remarks>To be added.</remarks>
         <exception cref="T:System.NotSupportedException">This method is not supported. This is a read-only collection.</exception>
       </Docs>
@@ -704,7 +704,7 @@
       </ReturnValue>
       <Parameters />
       <Docs>
-        <summary>Implementation from <see cref="T:System.Collections.Generic.ICollection" />. This method is not supported.</summary>
+        <summary>Clears the collection. Calling this method always throws <see cref="T:System.NotSupportedException" />.</summary>
         <remarks>To be added.</remarks>
         <exception cref="T:System.NotSupportedException">This method is not supported. This is a read-only collection.</exception>
       </Docs>
@@ -741,7 +741,13 @@
         <param name="item">The group to locate in the group collection.</param>
         <summary>Determines whether the group collection contains a specific group item.</summary>
         <returns><see langword="true" /> if the group item is found in the group collection; <see langword="false" /> otherwise.</returns>
-        <remarks>To be added.</remarks>
+        <remarks>  
+          <format type="text/markdown"><![CDATA[  
+
+This member is an explicit interface member implementation. It can be used only when the <xref:System.Text.RegularExpressions.GroupCollection> instance is cast to an <xref:System.Collections.Generic.ICollection%601> interface. 
+
+         ]]></format>
+         </remarks>
       </Docs>
     </Member>
     <Member MemberName="System.Collections.Generic.ICollection&lt;System.Text.RegularExpressions.Group&gt;.Remove">
@@ -773,9 +779,9 @@
         <Parameter Name="item" Type="System.Text.RegularExpressions.Group" Index="0" FrameworkAlternate="netcore-2.0;netcore-2.1;netcore-2.2;netcore-3.0;netstandard-2.1" />
       </Parameters>
       <Docs>
-        <param name="item">This method is not supported.</param>
-        <summary>Implementation from <see cref="T:System.Collections.Generic.ICollection" />. This method is not supported.</summary>
-        <returns>This method is not supported.</returns>
+        <param name="item">The group to remove.</param>
+        <summary>Removes a <see cref="T:System.Text.RegularExpressions.Group" /> from the collection. Calling this method always throws <see cref="T:System.NotSupportedException" />.</summary>
+        <returns>This method is not supported. No value is returned.</returns>
         <remarks>To be added.</remarks>
         <exception cref="T:System.NotSupportedException">This method is not supported. This is a read-only collection.</exception>
       </Docs>
@@ -806,14 +812,15 @@
       <Parameters />
       <Docs>
         <summary>Provides an enumerator that iterates through the group collection.</summary>
-        <returns>An enumerator that contains all <see cref="T:System.Text.RegularExpressions.Group" /> objects in the group collection.</returns>
+        <returns>An enumerator that contains all names and objects in the <see cref="T:System.Text.RegularExpressions.Group" /> collection.</returns>
         <remarks>
           <format type="text/markdown">
             <![CDATA[
 
 ## Remarks
 
-Instead of calling the <xref:System.Text.RegularExpressions.GroupCollection.GetEnumerator%2A> method to retrieve an enumerator that lets you iterate through the <xref:System.Text.RegularExpressions.Group> objects in the collection, you should use the group iteration construct (such as `foreach` in C# and `For Each`…`Next` in Visual Basic) that is provided by your programming language.
+
+Instead of calling this method to retrieve an enumerator that lets you iterate through the <xref:System.Text.RegularExpressions.Group> objects in the collection, you should use the group iteration construct (such as `foreach` in C# and `For Each`…`Next` in Visual Basic) that is provided by your programming language.
 
           ]]>
           </format>
@@ -855,7 +862,7 @@ Instead of calling the <xref:System.Text.RegularExpressions.GroupCollection.GetE
 
 ## Remarks
 
-Instead of calling the <xref:System.Text.RegularExpressions.GroupCollection.GetEnumerator%2A> method to retrieve an enumerator that lets you iterate through the <xref:System.Text.RegularExpressions.Group> objects in the collection, you should use the group iteration construct (such as `foreach` in C# and `For Each`…`Next` in Visual Basic) that is provided by your programming language.
+Instead of calling this method to retrieve an enumerator that lets you iterate through the <xref:System.Text.RegularExpressions.Group> objects in the collection, you should use the group iteration construct (such as `foreach` in C# and `For Each`…`Next` in Visual Basic) that is provided by your programming language.
 
           ]]>
           </format>
@@ -894,7 +901,13 @@ Instead of calling the <xref:System.Text.RegularExpressions.GroupCollection.GetE
         <param name="item">The group to locate in the group collection.</param>
         <summary>Determines the index of a specific group in the group collection.</summary>
         <returns>The index of the <paramref. name="item" /> if found; otherwise, -1.</returns>
-        <remarks>To be added.</remarks>
+        <remarks>  
+          <format type="text/markdown"><![CDATA[  
+
+This member is an explicit interface member implementation. It can be used only when the <xref:System.Text.RegularExpressions.GroupCollection> instance is cast to an <xref:System.Collections.Generic.IList%601> interface.
+
+         ]]></format>
+         </remarks>
       </Docs>
     </Member>
     <Member MemberName="System.Collections.Generic.IList&lt;System.Text.RegularExpressions.Group&gt;.Insert">
@@ -927,9 +940,9 @@ Instead of calling the <xref:System.Text.RegularExpressions.GroupCollection.GetE
         <Parameter Name="item" Type="System.Text.RegularExpressions.Group" Index="1" FrameworkAlternate="netcore-2.0;netcore-2.1;netcore-2.2;netcore-3.0;netstandard-2.1" />
       </Parameters>
       <Docs>
-        <param name="index">This method is not supported.</param>
-        <param name="item">This method is not supported.</param>
-        <summary>Implementation from <see cref="T:System.Collections.Generic.IList" />. This method is not supported.</summary>
+        <param name="index">The position at which to insert the group.</param>
+        <param name="item">The group to insert.</param>
+        <summary>Inserts a <see cref="T:System.Text.RegularExpressions.Group" /> into the collection. Calling this method always throws <see cref="T:System.NotSupportedException" />.</summary>
         <remarks>To be added.</remarks>
         <exception cref="T:System.NotSupportedException">This method is not supported. This is a read-only collection.</exception>
       </Docs>
@@ -964,10 +977,16 @@ Instead of calling the <xref:System.Text.RegularExpressions.GroupCollection.GetE
         <Parameter Name="index" Type="System.Int32" Index="0" FrameworkAlternate="netcore-2.0;netcore-2.1;netcore-2.2;netcore-3.0;netstandard-2.1" />
       </Parameters>
       <Docs>
-        <param name="index">The zero-index position of the group in the group collection.</param>
-        <summary>Gets the group in the desired position.</summary>
+        <param name="index">The zero-based index of the group in the group collection.</param>
+        <summary>Gets the group at the specified position in the collection.</summary>
         <value>The group in the desired position.</value>
-        <remarks>To be added.</remarks>
+        <remarks>  
+          <format type="text/markdown"><![CDATA[  
+
+This member is an explicit interface member implementation. It can be used only when the <xref:System.Text.RegularExpressions.GroupCollection> instance is cast to an <xref:System.Collections.Generic.IList%601> interface.
+
+         ]]></format>
+         </remarks>
         <exception cref="T:System.NotSupportedException">Cannot set an item. This is a read-only collection.</exception>
       </Docs>
     </Member>