Feedback: Simplify type declaration

dotnet · shyamnamboodiripad · Feb 13, 2025 · Jan 23, 2025 · Feb 11, 2025 · Feb 11, 2025
commit c8aa7a79795b03dca4b7cf74a39296767aeec41e
@@ -15,7 +15,5 @@ public partial class RelevanceTruthAndCompletenessEvaluator
         IgnoreReadOnlyProperties = true,
         PropertyNamingPolicy = JsonKnownNamingPolicy.CamelCase)]
     [JsonSerializable(typeof(Rating))]
-    internal sealed partial class SerializerContext : JsonSerializerContext
-    {
-    }
+    internal sealed partial class SerializerContext : JsonSerializerContext;
 }
@@ -9,6 +9,4 @@ namespace Microsoft.Extensions.AI.Evaluation;
 /// </summary>
 /// <param name="name">The name of the <see cref="BooleanMetric"/>.</param>
 /// <param name="value">The value of the <see cref="BooleanMetric"/>.</param>
-public sealed class BooleanMetric(string name, bool? value = null) : EvaluationMetric<bool?>(name, value)
-{
-}
+public sealed class BooleanMetric(string name, bool? value = null) : EvaluationMetric<bool?>(name, value);
@@ -7,6 +7,4 @@ namespace Microsoft.Extensions.AI.Evaluation;
 /// A base class that represents additional contextual information (beyond that which is available in the conversation
 /// history) that an <see cref="IEvaluator"/> may need to accurately evaluate a supplied response. 
 /// </summary>
-public class EvaluationContext
-{
-}
+public class EvaluationContext;
@@ -20,6 +20,4 @@ namespace Microsoft.Extensions.AI.Evaluation;
 /// </remarks>
 /// <param name="name">The name of the <see cref="NumericMetric"/>.</param>
 /// <param name="value">The value of the <see cref="NumericMetric"/>.</param>
-public sealed class NumericMetric(string name, double? value = null) : EvaluationMetric<double?>(name, value)
-{
-}
+public sealed class NumericMetric(string name, double? value = null) : EvaluationMetric<double?>(name, value);
@@ -12,6 +12,4 @@ namespace Microsoft.Extensions.AI.Evaluation;
 /// </remarks>
 /// <param name="name">The name of the <see cref="StringMetric"/>.</param>
 /// <param name="value">The value of the <see cref="StringMetric"/>.</param>
-public sealed class StringMetric(string name, string? value = null) : EvaluationMetric<string>(name, value)
-{
-}
+public sealed class StringMetric(string name, string? value = null) : EvaluationMetric<string>(name, value);