Remove INameTrimmer.Version

This no longer makes sense to keep and enabling features by baseline version seems fiddly.
If we need to toggle features for newer versions, we can explicitly add a boolean config option.

Author: Exanite

sources/SilkTouch/SilkTouch/Mods/PrettifyNames.cs

@@ -1449,9 +1449,6 @@ private NameAffixConfiguration GetConfiguration(string category) =>
     /// </summary>
     private class NameAffixerEarlyTrimmer(PrettifyNamesAffixer affixer) : INameTrimmer
     {
-        /// <inheritdoc/>
-        public Version Version => new(3, 0);
-
         public void Trim(NameTrimmerContext context)
         {
             if (context.Container == null)
@@ -1482,9 +1479,6 @@ public void Trim(NameTrimmerContext context)
 
     private class PrettifyNamesTrimmer(NamePrettifier namePrettifier) : INameTrimmer
     {
-        /// <inheritdoc/>
-        public Version Version => new(3, 0);
-
         public void Trim(NameTrimmerContext context)
         {
             foreach (var (original, (primary, secondary)) in context.Names)
@@ -1514,9 +1508,6 @@ public void Trim(NameTrimmerContext context)
     /// </summary>
     private class NameAffixerLateTrimmer(PrettifyNamesAffixer affixer) : INameTrimmer
     {
-        /// <inheritdoc/>
-        public Version Version => new(3, 0);
-
         public void Trim(NameTrimmerContext context)
         {
             if (context.Container == null)

sources/SilkTouch/SilkTouch/Naming/INameTrimmer.cs

@@ -5,16 +5,9 @@
 /// </summary>
 public interface INameTrimmer
 {
-    /// <summary>
-    /// Gets the name trimmer version i.e. the Silk.NET version that introduced this name trimming method. This is so
-    /// new bindings released with or after this version don't have to have older trimming methods executed (the name
-    /// version can be specified in the mod configuration)
-    /// </summary>
-    Version Version { get; }
-
     /// <summary>
     /// Trims prefixes from the given constituent names within the given container.
     /// </summary>
     /// <param name="context">The arguments.</param>
-    void Trim(NameTrimmerContext context);
+    public void Trim(NameTrimmerContext context);
 }

sources/SilkTouch/SilkTouch/Naming/NameTrimmer.cs

@@ -1,19 +1,10 @@
-using System;
-using System.Collections.Generic;
-using System.Diagnostics;
-using System.Linq;
-using System.Text.Json;
-
-namespace Silk.NET.SilkTouch.Naming;
+namespace Silk.NET.SilkTouch.Naming;
 
 /// <summary>
 /// The default name trimmer ported from Silk.NET 2.18 with modifications for 3.0.
 /// </summary>
 public class NameTrimmer : INameTrimmer
 {
-    /// <inheritdoc />
-    public virtual Version Version => new(3, 0);
-
     /// <summary>
     /// Determines whether a second pass without using <see cref="GetTrimmingName"/> is warranted.
     /// </summary>

sources/SilkTouch/SilkTouch/Naming/NameTrimmer217.cs

@@ -1,6 +1,4 @@
-using System;
-using System.Collections.Generic;
-using Humanizer;
+using Humanizer;
 
 namespace Silk.NET.SilkTouch.Naming;
 
@@ -10,9 +8,6 @@ namespace Silk.NET.SilkTouch.Naming;
 [Obsolete("Use NameTrimmer instead for 3.0 names")]
 public class NameTrimmer217 : NameTrimmer
 {
-    /// <inheritdoc />
-    public override Version Version => new(2, 17);
-
     /// <inheritdoc />
     protected override bool HasRawPass => false;
 

sources/SilkTouch/SilkTouch/Naming/NameTrimmer218.cs

@@ -1,16 +1,11 @@
-using System;
-
-namespace Silk.NET.SilkTouch.Naming;
+namespace Silk.NET.SilkTouch.Naming;
 
 /// <summary>
 /// The 2.18 name trimmer.
 /// </summary>
 [Obsolete("Use NameTrimmer instead for 3.0 names")]
 public class NameTrimmer218 : NameTrimmer
 {
-    /// <inheritdoc />
-    public override Version Version => new(2, 18);
-
     /// <inheritdoc />
     protected override bool HasRawPass => false;