Merge pull request #9 from configcat/feature/semverupdate

move semver into repo and remove nuget reference

LICENSE

@@ -12,6 +12,11 @@ furnished to do so, subject to the following conditions:
 The above copyright notice and this permission notice shall be included in all
 copies or substantial portions of the Software.
 
+This repository includes two files originally copied from https://github.com/maxhauser/semver
+repository: https://github.com/maxhauser/semver/blob/master/Semver/IntExtensions.cs and
+https://github.com/maxhauser/semver/blob/master/Semver/SemVersion.cs. The
+original versions of the files are MIT licensed. See the file headers for details.
+
 THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE

appveyor.yml

@@ -1,4 +1,4 @@
-version: 5.0.{build}
+version: 5.1.{build}
 image: Visual Studio 2019
 configuration: Release
 platform: Any CPU
@@ -22,7 +22,7 @@ before_test:
  - ps: choco install codecov --no-progress
 test_script:
  - ps: . .\set-debug-type.ps1 src\ConfigCatClient\ConfigCatClient.csproj
- - OpenCover.Console.exe -register:user -target:dotnet.exe -targetargs:"test src\ConfigCat.Client.Tests\ConfigCat.Client.Tests.csproj -c Release" -output:.\coverage.xml -filter:"+[*]ConfigCat.Client.* -[ConfigCatClientTests]*" -oldstyle -returntargetcode
+ - cmd: coverage.cmd
 after_test:
  - codecov -f "coverage.xml"
 artifacts:
@@ -33,4 +33,4 @@ notifications:
   to:
   - [email protected]
   on_build_success: false
-  on_build_failure: true
+  on_build_failure: true

coverage.cmd

@@ -1,3 +1 @@
-OpenCover.Console.exe -register:user -target:dotnet.exe -targetargs:"test src\ConfigCat.Client.Tests\ConfigCat.Client.Tests.csproj -c Release" -output:.\coverage.xml -filter:"+[*]ConfigCat.Client.* -[ConfigCatClientTests]*" -oldstyle -returntargetcode
-
-pause
+OpenCover.Console.exe -register:user -target:dotnet.exe -targetargs:"test src\ConfigCat.Client.Tests\ConfigCat.Client.Tests.csproj -c Release" -output:.\coverage.xml -filter:"+[*]ConfigCat.Client.* -[ConfigCatClientTests]* -[*]ConfigCat.Client.Versioning.*" -oldstyle -returntargetcode

src/ConfigCatClient/ConfigCatClient.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
      <PropertyGroup>
-          <TargetFrameworks>netstandard1.3;netstandard2.0</TargetFrameworks>
+          <TargetFrameworks>net45;netstandard1.3;netstandard2.0</TargetFrameworks>
           <AssemblyName>ConfigCat.Client</AssemblyName>
           <RootNamespace>ConfigCat.Client</RootNamespace>
           <SignAssembly>true</SignAssembly>
@@ -15,7 +15,9 @@
           <PackageProjectUrl>https://github.com/ConfigCat/.net-sdk</PackageProjectUrl>
           <RepositoryUrl>https://github.com/ConfigCat/.net-sdk</RepositoryUrl>
           <RepositoryType>git</RepositoryType>
-          <PackageReleaseNotes>Version 5.0.0
+          <PackageReleaseNotes>Version 5.1.0
+     * Remove semver nuget packages
+Version 5.0.0
      * Breaking change: Renamed `API Key` to `SDK Key`.
 Version 4.0.0
      * Supporting sensitive text comparators.
@@ -77,7 +79,6 @@ Works with .NET, .NET Core, .NET Standard</Description>
           <PackageReference Include="Newtonsoft.Json" Version="10.0.3" />
           <PackageReference Include="System.Net.Http" Version="4.3.4" />
           <PackageReference Include="System.Text.RegularExpressions" Version="4.3.1" />
-          <PackageReference Include="semver" Version="2.0.4" />
      </ItemGroup>
 
 </Project>

src/ConfigCatClient/Evaluate/RolloutEvaluator.cs

@@ -1,5 +1,5 @@
-using Newtonsoft.Json.Linq;
-using Semver;
+using ConfigCat.Client.Versioning;
+using Newtonsoft.Json.Linq;
 using System;
 using System.Collections.Generic;
 using System.Globalization;

src/ConfigCatClient/Versioning/IntExtensions.cs

@@ -0,0 +1,55 @@
+//Copyright(c) 2013 Max Hauser
+
+//Permission is hereby granted, free of charge, to any person obtaining a copy
+//of this software and associated documentation files (the "Software"), to deal
+//in the Software without restriction, including without limitation the rights
+//to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+//copies of the Software, and to permit persons to whom the Software is
+//furnished to do so, subject to the following conditions:
+
+//The above copyright notice and this permission notice shall be included in
+//all copies or substantial portions of the Software.
+
+//THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+//IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+//FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+//AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+//LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+//OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+//THE SOFTWARE.
+
+using System.Text;
+#if !NETSTANDARD
+#endif
+
+namespace ConfigCat.Client.Versioning
+{
+    internal static class IntExtensions
+    {
+        /// <summary>
+        /// The number of digits in a non-negative number. Returns 1 for all
+        /// negative numbers. That is ok because we are using it to calculate
+        /// string length for a <see cref="StringBuilder"/> for numbers that
+        /// aren't supposed to be negative, but when they are it is just a little
+        /// slower.
+        /// </summary>
+        /// <remarks>
+        /// This approach is based on https://stackoverflow.com/a/51099524/268898
+        /// where the poster offers performance benchmarks showing this is the
+        /// fastest way to get a number of digits.
+        /// </remarks>
+        public static int Digits(this int n)
+        {
+            if (n < 10) return 1;
+            if (n < 100) return 2;
+            if (n < 1_000) return 3;
+            if (n < 10_000) return 4;
+            if (n < 100_000) return 5;
+            if (n < 1_000_000) return 6;
+            if (n < 10_000_000) return 7;
+            if (n < 100_000_000) return 8;
+            if (n < 1_000_000_000) return 9;
+            return 10;
+        }
+    }
+}