Clean up suppressions

Author: RehanSaeed

Source/Common/HashCode.cs

@@ -114,9 +114,7 @@ public override bool Equals(object? obj)
         [EditorBrowsable(EditorBrowsableState.Never)]
         public override int GetHashCode() =>
 #pragma warning disable CA1065 // Do not raise exceptions in unexpected locations
-#pragma warning disable CA1303 // Do not pass literals as localized parameters
             throw new NotSupportedException("Implicitly convert this struct to an int to get the hash code.");
-#pragma warning restore CA1303 // Do not pass literals as localized parameters
 #pragma warning restore CA1065 // Do not raise exceptions in unexpected locations
 
         private static int CombineHashCodes(int h1, int h2)

Source/Common/OneOrMany{T}.cs

@@ -10,10 +10,8 @@ namespace Schema.NET
     /// </summary>
     /// <typeparam name="T">The type of the values.</typeparam>
     /// <seealso cref="ICollection{T}" />
-#pragma warning disable CA1710 // Identifiers should have correct suffix
     public readonly struct OneOrMany<T>
         : IReadOnlyCollection<T>, IEnumerable<T>, IValues, IEquatable<OneOrMany<T>>
-#pragma warning restore CA1710 // Identifiers should have correct suffix
     {
         private readonly T[]? collection;
 

Source/Common/Thing.Partial.cs

@@ -5,9 +5,7 @@ namespace Schema.NET
     /// <summary>
     /// The most generic type of item.
     /// </summary>
-#pragma warning disable CA1040 // Avoid empty interfaces
     public partial interface IThing
-#pragma warning restore CA1040 // Avoid empty interfaces
     {
     }
 

Source/Common/Values{T1,T2,T3,T4,T5,T6,T7}.cs

@@ -11,10 +11,8 @@ namespace Schema.NET
     /// <typeparam name="T1">The first type the values can take.</typeparam>
     /// <typeparam name="T2">The second type the values can take.</typeparam>
     /// <typeparam name="T3">The third type the values can take.</typeparam>
-#pragma warning disable CA1710 // Identifiers should have correct suffix
     public readonly struct Values<T1, T2, T3>
         : IReadOnlyCollection<object?>, IEnumerable<object?>, IValues, IEquatable<Values<T1, T2, T3>>
-#pragma warning restore CA1710 // Identifiers should have correct suffix
     {
         /// <summary>
         /// Initializes a new instance of the <see cref="Values{T1,T2,T3}"/> struct.
@@ -162,112 +160,84 @@ public Values(IEnumerable<object> items)
         /// </summary>
         public OneOrMany<T3> Value3 { get; }
 
+#pragma warning disable CA1002 // Do not expose generic lists
+#pragma warning disable CA2225 // Operator overloads have named alternates
         /// <summary>
         /// Performs an implicit conversion from <typeparamref name="T1"/> to <see cref="Values{T1,T2}"/>.
         /// </summary>
         /// <param name="item">The single item value.</param>
         /// <returns>The result of the conversion.</returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator Values<T1, T2, T3>(T1 item) => new(item);
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <typeparamref name="T2"/> to <see cref="Values{T1,T2}"/>.
         /// </summary>
         /// <param name="item">The single item value.</param>
         /// <returns>The result of the conversion.</returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator Values<T1, T2, T3>(T2 item) => new(item);
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <typeparamref name="T3"/> to <see cref="Values{T1,T2}"/>.
         /// </summary>
         /// <param name="item">The single item value.</param>
         /// <returns>The result of the conversion.</returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator Values<T1, T2, T3>(T3 item) => new(item);
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <typeparamref name="T1[]"/> to <see cref="Values{T1,T2,T3}"/>.
         /// </summary>
         /// <param name="array">The array of values.</param>
         /// <returns>The result of the conversion.</returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator Values<T1, T2, T3>(T1[] array) => new(array);
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <typeparamref name="T2[]"/> to <see cref="Values{T1,T2,T3}"/>.
         /// </summary>
         /// <param name="array">The array of values.</param>
         /// <returns>The result of the conversion.</returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator Values<T1, T2, T3>(T2[] array) => new(array);
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <typeparamref name="T3[]"/> to <see cref="Values{T1,T2,T3}"/>.
         /// </summary>
         /// <param name="array">The array of values.</param>
         /// <returns>The result of the conversion.</returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator Values<T1, T2, T3>(T3[] array) => new(array);
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="List{T1}"/> to <see cref="Values{T1,T2}"/>.
         /// </summary>
         /// <param name="list">The list of values.</param>
         /// <returns>The result of the conversion.</returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
-#pragma warning disable CA1002 // Do not expose generic lists
         public static implicit operator Values<T1, T2, T3>(List<T1> list) => new(list);
-#pragma warning restore CA1002 // Do not expose generic lists
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="List{T2}"/> to <see cref="Values{T1,T2}"/>.
         /// </summary>
         /// <param name="list">The list of values.</param>
         /// <returns>The result of the conversion.</returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
-#pragma warning disable CA1002 // Do not expose generic lists
         public static implicit operator Values<T1, T2, T3>(List<T2> list) => new(list);
-#pragma warning restore CA1002 // Do not expose generic lists
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="List{T3}"/> to <see cref="Values{T1,T2}"/>.
         /// </summary>
         /// <param name="list">The list of values.</param>
         /// <returns>The result of the conversion.</returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
-#pragma warning disable CA1002 // Do not expose generic lists
         public static implicit operator Values<T1, T2, T3>(List<T3> list) => new(list);
-#pragma warning restore CA1002 // Do not expose generic lists
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="object"/> array to <see cref="Values{T1,T2,T3}"/>.
         /// </summary>
         /// <param name="array">The array of values.</param>
         /// <returns>The result of the conversion.</returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator Values<T1, T2, T3>(object[] array) => new(array);
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="List{Object}"/> to <see cref="Values{T1,T2,T3}"/>.
         /// </summary>
         /// <param name="list">The list of values.</param>
         /// <returns>The result of the conversion.</returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
-#pragma warning disable CA1002 // Do not expose generic lists
         public static implicit operator Values<T1, T2, T3>(List<object> list) => new(list);
-#pragma warning restore CA1002 // Do not expose generic lists
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="Values{T1, T2, T3}"/> to the first item of type <typeparamref name="T1"/>.
@@ -276,9 +246,7 @@ public Values(IEnumerable<object> items)
         /// <returns>
         /// The result of the conversion.
         /// </returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator T1?(Values<T1, T2, T3> values) => values.Value1.FirstOrDefault();
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="Values{T1, T2, T3}"/> to the first item of type <typeparamref name="T2"/>.
@@ -287,9 +255,7 @@ public Values(IEnumerable<object> items)
         /// <returns>
         /// The result of the conversion.
         /// </returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator T2?(Values<T1, T2, T3> values) => values.Value2.FirstOrDefault();
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="Values{T1, T2, T3}"/> to the first item of type <typeparamref name="T3"/>.
@@ -298,9 +264,7 @@ public Values(IEnumerable<object> items)
         /// <returns>
         /// The result of the conversion.
         /// </returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator T3?(Values<T1, T2, T3> values) => values.Value3.FirstOrDefault();
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="Values{T1, T2, T3}"/> to an array of <typeparamref name="T1"/>.
@@ -309,9 +273,7 @@ public Values(IEnumerable<object> items)
         /// <returns>
         /// The result of the conversion.
         /// </returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator T1[](Values<T1, T2, T3> values) => values.Value1.ToArray();
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="Values{T1, T2, T3}"/> to <see cref="List{T1}"/>.
@@ -320,11 +282,7 @@ public Values(IEnumerable<object> items)
         /// <returns>
         /// The result of the conversion.
         /// </returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
-#pragma warning disable CA1002 // Do not expose generic lists
         public static implicit operator List<T1>(Values<T1, T2, T3> values) => values.Value1.ToList();
-#pragma warning restore CA1002 // Do not expose generic lists
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="Values{T1, T2, T3}"/> to an array of <typeparamref name="T2"/>.
@@ -333,9 +291,7 @@ public Values(IEnumerable<object> items)
         /// <returns>
         /// The result of the conversion.
         /// </returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator T2[](Values<T1, T2, T3> values) => values.Value2.ToArray();
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="Values{T1, T2, T3}"/> to <see cref="List{T2}"/>.
@@ -344,11 +300,7 @@ public Values(IEnumerable<object> items)
         /// <returns>
         /// The result of the conversion.
         /// </returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
-#pragma warning disable CA1002 // Do not expose generic lists
         public static implicit operator List<T2>(Values<T1, T2, T3> values) => values.Value2.ToList();
-#pragma warning restore CA1002 // Do not expose generic lists
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="Values{T1, T2, T3}"/> to an array of <typeparamref name="T3"/>.
@@ -357,9 +309,7 @@ public Values(IEnumerable<object> items)
         /// <returns>
         /// The result of the conversion.
         /// </returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
         public static implicit operator T3[](Values<T1, T2, T3> values) => values.Value3.ToArray();
-#pragma warning restore CA2225 // Operator overloads have named alternates
 
         /// <summary>
         /// Performs an implicit conversion from <see cref="Values{T1, T2, T3}"/> to <see cref="List{T3}"/>.
@@ -368,11 +318,9 @@ public Values(IEnumerable<object> items)
         /// <returns>
         /// The result of the conversion.
         /// </returns>
-#pragma warning disable CA2225 // Operator overloads have named alternates
-#pragma warning disable CA1002 // Do not expose generic lists
         public static implicit operator List<T3>(Values<T1, T2, T3> values) => values.Value3.ToList();
-#pragma warning restore CA1002 // Do not expose generic lists
 #pragma warning restore CA2225 // Operator overloads have named alternates
+#pragma warning restore CA1002 // Do not expose generic lists
 
         /// <summary>
         /// Implements the operator ==.