Skip to content

Commit

Permalink
Merge branch 'master' into expand-documentation
Browse files Browse the repository at this point in the history
  • Loading branch information
wallymathieu committed Jan 24, 2021
2 parents 8c8a9c0 + 6263a31 commit 3686417
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 16 deletions.
8 changes: 8 additions & 0 deletions docsrc/content/giraffe.fsx
Original file line number Diff line number Diff line change
Expand Up @@ -83,3 +83,11 @@ let personHandler =
return! json person next ctx
// and ideally we would deal with case when the parsing fails as well
}

(**
The benefit of doing an integration in this way is:
- You get a compilation error when trying to use types that don't have the proper functions defined.
- You avoid having to use runtime reflection to bind and serialize the Json.
- You have more control over the serialization than with System.Text.Json or Newtonsoft.Json
*)
4 changes: 0 additions & 4 deletions src/Fleece.FSharpData/Fleece.FSharpData.fsproj
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,5 @@
<PackageReference Update="FSharp.Core" Version="4.6.2" />
<PackageReference Include="FSharpPlus" Version="1.1.1" />
<PackageReference Include="FSharp.Data" Version="3.0.0" />
<PackageReference Include="Microsoft.NETFramework.ReferenceAssemblies" Version="1.0.0">
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
</PackageReference>
</ItemGroup>
</Project>
4 changes: 0 additions & 4 deletions src/Fleece.NewtonsoftJson/Fleece.NewtonsoftJson.fsproj
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,5 @@
<PackageReference Update="FSharp.Core" Version="4.6.2" />
<PackageReference Include="FSharpPlus" Version="1.1.1" />
<PackageReference Include="Newtonsoft.Json" Version="10.0.2" />
<PackageReference Include="Microsoft.NETFramework.ReferenceAssemblies" Version="1.0.0">
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
</PackageReference>
</ItemGroup>
</Project>
4 changes: 0 additions & 4 deletions src/Fleece.SystemJson/Fleece.SystemJson.fsproj
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,5 @@
<PackageReference Update="FSharp.Core" Version="4.6.2" />
<PackageReference Include="FSharpPlus" Version="1.1.1" />
<PackageReference Include="System.Json" Version="4.7.1" />
<PackageReference Include="Microsoft.NETFramework.ReferenceAssemblies" Version="1.0.0">
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
</PackageReference>
</ItemGroup>
</Project>
4 changes: 0 additions & 4 deletions src/Fleece.SystemTextJson/Fleece.SystemTextJson.fsproj
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,5 @@
<PackageReference Update="FSharp.Core" Version="4.6.2" />
<PackageReference Include="FSharpPlus" Version="1.1.1" />
<PackageReference Include="System.Text.Json" Version="4.7.1" />
<PackageReference Include="Microsoft.NETFramework.ReferenceAssemblies" Version="1.0.0">
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
</PackageReference>
</ItemGroup>
</Project>

0 comments on commit 3686417

Please sign in to comment.