Restore NonVersionableAttribute for SystemCollectionsImmutable

Author: andrewlock

tracer/build/_build/UpdateVendors/VendoredDependency.cs

@@ -393,7 +393,6 @@ private static string FixSystemCollectionsImmutable(string filePath, string cont
             contents = contents.Insert(namespaceIndex, usings);
 
             contents = contents
-               .Replace("[NonVersionable]", "/* [NonVersionable] */")
                .Replace("[ValidatedNotNull]", string.Empty)
                .Replace("#nullable restore", "#nullable enable");
 

tracer/src/Datadog.Trace/Vendors/System.Collections.Immutable/System/Collections/Immutable/ImmutableArray_1.Minimal.cs

@@ -47,7 +47,7 @@ namespace Datadog.Trace.VendoredMicrosoftCode.System.Collections.Immutable
     /// it is insulated from other threads.
     /// </devremarks>
     [DebuggerDisplay("{DebuggerDisplay,nq}")]
-    /* [NonVersionable] */ // Applies to field layout
+    [NonVersionable] // Applies to field layout
     internal partial struct ImmutableArray<T> : IEnumerable<T>, IEquatable<ImmutableArray<T>>, IImmutableArray
     {
         /// <summary>
@@ -86,7 +86,7 @@ internal ImmutableArray(T[]? items)
         /// <param name="left">The instance to the left of the operator.</param>
         /// <param name="right">The instance to the right of the operator.</param>
         /// <returns><c>true</c> if the values' underlying arrays are reference equal; <c>false</c> otherwise.</returns>
-        /* [NonVersionable] */
+        [NonVersionable]
         public static bool operator ==(ImmutableArray<T> left, ImmutableArray<T> right)
         {
             return left.Equals(right);
@@ -98,7 +98,7 @@ internal ImmutableArray(T[]? items)
         /// <param name="left">The instance to the left of the operator.</param>
         /// <param name="right">The instance to the right of the operator.</param>
         /// <returns><c>true</c> if the values' underlying arrays are reference not equal; <c>false</c> otherwise.</returns>
-        /* [NonVersionable] */
+        [NonVersionable]
         public static bool operator !=(ImmutableArray<T> left, ImmutableArray<T> right)
         {
             return !left.Equals(right);
@@ -135,7 +135,7 @@ internal ImmutableArray(T[]? items)
         /// <returns>The element at the specified index in the read-only list.</returns>
         public T this[int index]
         {
-            /* [NonVersionable] */
+            [NonVersionable]
             get
             {
                 // We intentionally do not check this.array != null, and throw NullReferenceException
@@ -168,7 +168,7 @@ public ref readonly T ItemRef(int index)
         [DebuggerBrowsable(DebuggerBrowsableState.Never)]
         public bool IsEmpty
         {
-            /* [NonVersionable] */
+            [NonVersionable]
             get { return this.array!.Length == 0; }
         }
 
@@ -178,7 +178,7 @@ public bool IsEmpty
         [DebuggerBrowsable(DebuggerBrowsableState.Never)]
         public int Length
         {
-            /* [NonVersionable] */
+            [NonVersionable]
             get
             {
                 // We intentionally do not check this.array != null, and throw NullReferenceException
@@ -331,7 +331,7 @@ public override bool Equals([NotNullWhen(true)] object? obj)
         /// <returns>
         /// true if the current object is equal to the <paramref name="other"/> parameter; otherwise, false.
         /// </returns>
-        /* [NonVersionable] */
+        [NonVersionable]
         public bool Equals(ImmutableArray<T> other)
         {
             return this.array == other.array;