Skip to content

Commit

Permalink
Upgrade Polyfill and C# lang version
Browse files Browse the repository at this point in the history
Polyfill fixes the Index/Range issue in 1.33.2
  • Loading branch information
SadPencil committed Dec 10, 2023
1 parent e819213 commit 47f779b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

<PropertyGroup>
<TargetFramework>netstandard2.0</TargetFramework>
<LangVersion>10.0</LangVersion>
<LangVersion>12.0</LangVersion>
</PropertyGroup>

<ItemGroup>
Expand Down
6 changes: 2 additions & 4 deletions build/WinForms.props
Original file line number Diff line number Diff line change
@@ -1,20 +1,18 @@
<Project>
<PropertyGroup>
<LangVersion>11.0</LangVersion>
<LangVersion>12.0</LangVersion>
</PropertyGroup>
<PropertyGroup Condition="'$(Engine)' != 'UniversalGL'">
<TargetFramework>net48</TargetFramework>
<UseWindowsForms>true</UseWindowsForms>
<!-- workaround: temporarily bypass Polyfill's missing Index Range issue -->
<DefineConstants>$(DefineConstants);NET48X</DefineConstants>
</PropertyGroup>
<PropertyGroup Condition="'$(Engine)' == 'UniversalGL'">
<TargetFramework>net7.0</TargetFramework>
<UseWindowsForms>false</UseWindowsForms>
</PropertyGroup>

<ItemGroup Condition="'$(Engine)' != 'UniversalGL'">
<PackageReference Include="Polyfill" Version="1.32.0">
<PackageReference Include="Polyfill" Version="1.33.2">
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
</PackageReference>
Expand Down

0 comments on commit 47f779b

Please sign in to comment.