Skip to content

Commit

Permalink
Merge pull request #224 from bartelink/static-state-machine-wae
Browse files Browse the repository at this point in the history
Enable `TreatWarningsAsErrors` across the board
  • Loading branch information
abelbraaksma authored Dec 24, 2023
2 parents 9186f71 + 6a743a7 commit 2692bb1
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

<PropertyGroup>
<TargetFramework>net6.0</TargetFramework>
<TreatWarningsAsErrors>True</TreatWarningsAsErrors>
</PropertyGroup>

<ItemGroup>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
<Project Sdk="Microsoft.NET.Sdk">
<Project Sdk="Microsoft.NET.Sdk">

<PropertyGroup>
<TargetFramework>net6.0</TargetFramework>
<TreatWarningsAsErrors>True</TreatWarningsAsErrors>
</PropertyGroup>

<ItemGroup>
Expand Down
11 changes: 6 additions & 5 deletions src/FSharp.Control.TaskSeq.Test/TaskSeq.MaxMin.Tests.fs
Original file line number Diff line number Diff line change
Expand Up @@ -296,7 +296,7 @@ module SideSeffects =
}

[<Theory; ClassData(typeof<JustMinMaxBy>)>]
let ``TaskSeq-minBy, maxBy with sequence that changes length`` (minMax: MinMax) = task {
let ``TaskSeq-minBy, maxBy with sequence that changes length`` (minMax: MinMax) =
let mutable i = 0

let ts = taskSeq {
Expand All @@ -311,7 +311,8 @@ module SideSeffects =
else
minMaxFn id ts |> Task.map (should equal v)

do! test (MinMax.getByFunction minMax) 10
do! test (MinMax.getByFunction minMax) 20
do! test (MinMax.getByFunction minMax) 30
}
task {
do! test (MinMax.getByFunction minMax) 10
do! test (MinMax.getByFunction minMax) 20
do! test (MinMax.getByFunction minMax) 30
}
3 changes: 1 addition & 2 deletions src/FSharp.Control.TaskSeq/FSharp.Control.TaskSeq.fsproj
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

<PropertyGroup>
<TargetFramework>netstandard2.1</TargetFramework>
<TreatWarningsAsErrors>True</TreatWarningsAsErrors>
<GenerateDocumentationFile>true</GenerateDocumentationFile>
<GeneratePackageOnBuild>True</GeneratePackageOnBuild>
<Title>Computation expression 'taskSeq' for processing IAsyncEnumerable sequences and module functions</Title>
Expand All @@ -27,11 +28,9 @@ Generates optimized IL code through resumable state machines, and comes with a c
</PropertyGroup>

<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
<OtherFlags></OtherFlags>
</PropertyGroup>

<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|AnyCPU'">
<OtherFlags></OtherFlags>
<Tailcalls>True</Tailcalls>
</PropertyGroup>

Expand Down

0 comments on commit 2692bb1

Please sign in to comment.