Github user FlorianHockmann commented on a diff in the pull request:

    https://github.com/apache/tinkerpop/pull/842#discussion_r209602159
  
    --- Diff: gremlin-dotnet/src/Gremlin.Net/Gremlin.Net.csproj ---
    @@ -48,7 +48,7 @@ Please see the reference documentation at Apache 
TinkerPop for more information
       </PropertyGroup>
     
       <ItemGroup>
    -    <PackageReference Include="Newtonsoft.Json" Version="9.0.1" />
    +    <PackageReference Include="Newtonsoft.Json" Version="11.0.2" />
    --- End diff --
    
    The reason was that this version of Newtonsoft.Json couldn't deserialize 
_TooBigInteger_ according to @danielcweber and I didn't see a good reason why 
we shouldn't update the dependency.
    Are you against the version bump?


---

Reply via email to