Skip to content

Commit dfbc3ee

Browse files
askptCopilot
andauthored
refactor: Clean up project files by removing TargetFrameworks and formatting (#611)
* refactor: Clean up project files by removing TargetFrameworks and formatting Signed-off-by: André Silva <[email protected]> * Update src/Directory.Build.props Co-authored-by: Copilot <[email protected]> Signed-off-by: André Silva <[email protected]> --------- Signed-off-by: André Silva <[email protected]> Co-authored-by: Copilot <[email protected]>
1 parent 29e1657 commit dfbc3ee

File tree

5 files changed

+78
-75
lines changed

5 files changed

+78
-75
lines changed

src/Directory.Build.props

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,10 @@
11
<Project>
2-
<Import Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(MSBuildProjectDirectory), 'OpenFeature.slnx'))\build\Common.prod.props" />
2+
<Import
3+
Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(MSBuildProjectDirectory), 'OpenFeature.slnx'))\build\Common.prod.props" />
34
<PropertyGroup>
4-
<!-- Enable native AOT related analyzers https://learn.microsoft.com/dotnet/core/deploying/native-aot/#aot-compatibility-analyzers -->
5+
<TargetFrameworks>net462;netstandard2.0;net8.0;net9.0</TargetFrameworks>
6+
<!-- Enable native AOT related analyzers
7+
https://learn.microsoft.com/dotnet/core/deploying/native-aot/#aot-compatibility-analyzers -->
58
<IsAotCompatible>$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'net8.0'))</IsAotCompatible>
69
</PropertyGroup>
7-
</Project>
10+
</Project>
Lines changed: 17 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,23 @@
11
<Project Sdk="Microsoft.NET.Sdk">
22

3-
<PropertyGroup>
4-
<TargetFrameworks>netstandard2.0;net8.0;net9.0;net462</TargetFrameworks>
5-
<RootNamespace>OpenFeature.DependencyInjection</RootNamespace>
6-
<PackageReadmeFile>README.md</PackageReadmeFile>
7-
</PropertyGroup>
3+
<PropertyGroup>
4+
<RootNamespace>OpenFeature.DependencyInjection</RootNamespace>
5+
<PackageReadmeFile>README.md</PackageReadmeFile>
6+
</PropertyGroup>
87

9-
<ItemGroup>
10-
<PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" />
11-
<PackageReference Include="Microsoft.Extensions.Options" />
12-
</ItemGroup>
8+
<ItemGroup>
9+
<PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" />
10+
<PackageReference Include="Microsoft.Extensions.Options" />
11+
</ItemGroup>
1312

14-
<ItemGroup>
15-
<ProjectReference Include="..\OpenFeature\OpenFeature.csproj" />
16-
</ItemGroup>
13+
<ItemGroup>
14+
<ProjectReference Include="..\OpenFeature\OpenFeature.csproj" />
15+
</ItemGroup>
1716

18-
<ItemGroup>
19-
<InternalsVisibleTo Include="$(AssemblyName).Tests" />
20-
<InternalsVisibleTo Include="DynamicProxyGenAssembly2" />
21-
<None Include="README.md" Pack="true" PackagePath="/" />
22-
</ItemGroup>
17+
<ItemGroup>
18+
<InternalsVisibleTo Include="$(AssemblyName).Tests" />
19+
<InternalsVisibleTo Include="DynamicProxyGenAssembly2" />
20+
<None Include="README.md" Pack="true" PackagePath="/" />
21+
</ItemGroup>
2322

24-
</Project>
23+
</Project>
Lines changed: 16 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,22 @@
11
<Project Sdk="Microsoft.NET.Sdk">
22

3-
<PropertyGroup>
4-
<TargetFrameworks>netstandard2.0;net8.0;net9.0;net462</TargetFrameworks>
5-
<RootNamespace>OpenFeature</RootNamespace>
6-
<PackageReadmeFile>README.md</PackageReadmeFile>
7-
</PropertyGroup>
3+
<PropertyGroup>
4+
<RootNamespace>OpenFeature</RootNamespace>
5+
<PackageReadmeFile>README.md</PackageReadmeFile>
6+
</PropertyGroup>
87

9-
<ItemGroup>
10-
<PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" />
11-
</ItemGroup>
8+
<ItemGroup>
9+
<PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" />
10+
</ItemGroup>
1211

13-
<ItemGroup>
14-
<ProjectReference Include="..\OpenFeature\OpenFeature.csproj" />
15-
</ItemGroup>
12+
<ItemGroup>
13+
<ProjectReference Include="..\OpenFeature\OpenFeature.csproj" />
14+
</ItemGroup>
1615

17-
<ItemGroup>
18-
<InternalsVisibleTo Include="$(AssemblyName).Tests" />
19-
<InternalsVisibleTo Include="DynamicProxyGenAssembly2" />
20-
<None Include="README.md" Pack="true" PackagePath="/" />
21-
</ItemGroup>
16+
<ItemGroup>
17+
<InternalsVisibleTo Include="$(AssemblyName).Tests" />
18+
<InternalsVisibleTo Include="DynamicProxyGenAssembly2" />
19+
<None Include="README.md" Pack="true" PackagePath="/" />
20+
</ItemGroup>
2221

23-
</Project>
22+
</Project>
Lines changed: 13 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,17 @@
11
<Project Sdk="Microsoft.NET.Sdk">
22

3-
<PropertyGroup>
4-
<TargetFrameworks>net8.0;net9.0;netstandard2.0;net462</TargetFrameworks>
5-
<RootNamespace>OpenFeature.Providers.MultiProvider</RootNamespace>
6-
<PackageReadmeFile>README.md</PackageReadmeFile>
7-
</PropertyGroup>
3+
<PropertyGroup>
4+
<RootNamespace>OpenFeature.Providers.MultiProvider</RootNamespace>
5+
<PackageReadmeFile>README.md</PackageReadmeFile>
6+
</PropertyGroup>
87

9-
<ItemGroup>
10-
<InternalsVisibleTo Include="DynamicProxyGenAssembly2" />
11-
<InternalsVisibleTo Include="OpenFeature.Providers.MultiProvider.Tests" />
12-
<None Include="README.md" Pack="true" PackagePath="/" />
13-
</ItemGroup>
8+
<ItemGroup>
9+
<InternalsVisibleTo Include="DynamicProxyGenAssembly2" />
10+
<InternalsVisibleTo Include="OpenFeature.Providers.MultiProvider.Tests" />
11+
<None Include="README.md" Pack="true" PackagePath="/" />
12+
</ItemGroup>
1413

15-
<ItemGroup>
16-
<ProjectReference Include="..\OpenFeature\OpenFeature.csproj" />
17-
</ItemGroup>
18-
</Project>
14+
<ItemGroup>
15+
<ProjectReference Include="..\OpenFeature\OpenFeature.csproj" />
16+
</ItemGroup>
17+
</Project>

src/OpenFeature/OpenFeature.csproj

Lines changed: 26 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,31 @@
11
<Project Sdk="Microsoft.NET.Sdk">
22

3-
<PropertyGroup>
4-
<TargetFrameworks>net8.0;net9.0;netstandard2.0;net462</TargetFrameworks>
5-
<RootNamespace>OpenFeature</RootNamespace>
6-
<PackageReadmeFile>README.md</PackageReadmeFile>
7-
</PropertyGroup>
3+
<PropertyGroup>
4+
<RootNamespace>OpenFeature</RootNamespace>
5+
<PackageReadmeFile>README.md</PackageReadmeFile>
6+
</PropertyGroup>
87

9-
<ItemGroup>
10-
<PackageReference Include="Microsoft.Bcl.AsyncInterfaces" Condition="'$(TargetFramework)' == 'net462' or '$(TargetFramework)' == 'netstandard2.0'" />
11-
<PackageReference Include="Microsoft.Bcl.HashCode" Condition="'$(TargetFramework)' == 'net462' or '$(TargetFramework)' == 'netstandard2.0'" />
12-
<PackageReference Include="Microsoft.Extensions.Logging.Abstractions" />
13-
<PackageReference Include="System.Diagnostics.DiagnosticSource" Condition="'$(TargetFramework)' == 'net462' or '$(TargetFramework)' == 'netstandard2.0'" />
14-
<PackageReference Include="System.Text.Json" Condition="'$(TargetFramework)' == 'net462' or '$(TargetFramework)' == 'netstandard2.0'" />
15-
</ItemGroup>
8+
<ItemGroup>
9+
<PackageReference Include="Microsoft.Bcl.AsyncInterfaces"
10+
Condition="'$(TargetFramework)' == 'net462' or '$(TargetFramework)' == 'netstandard2.0'" />
11+
<PackageReference Include="Microsoft.Bcl.HashCode"
12+
Condition="'$(TargetFramework)' == 'net462' or '$(TargetFramework)' == 'netstandard2.0'" />
13+
<PackageReference Include="Microsoft.Extensions.Logging.Abstractions" />
14+
<PackageReference Include="System.Diagnostics.DiagnosticSource"
15+
Condition="'$(TargetFramework)' == 'net462' or '$(TargetFramework)' == 'netstandard2.0'" />
16+
<PackageReference Include="System.Text.Json"
17+
Condition="'$(TargetFramework)' == 'net462' or '$(TargetFramework)' == 'netstandard2.0'" />
18+
</ItemGroup>
1619

17-
<ItemGroup>
18-
<InternalsVisibleTo Include="DynamicProxyGenAssembly2" />
19-
<InternalsVisibleTo Include="OpenFeature.Benchmarks" />
20-
<InternalsVisibleTo Include="OpenFeature.Tests" />
21-
<InternalsVisibleTo Include="OpenFeature.E2ETests" />
22-
<InternalsVisibleTo Include="OpenFeature.DependencyInjection" />
23-
<InternalsVisibleTo Include="OpenFeature.Providers.MultiProvider" />
24-
<InternalsVisibleTo Include="OpenFeature.Providers.MultiProvider.Tests" />
25-
<None Include="../../README.md" Pack="true" PackagePath="/" />
26-
</ItemGroup>
20+
<ItemGroup>
21+
<InternalsVisibleTo Include="DynamicProxyGenAssembly2" />
22+
<InternalsVisibleTo Include="OpenFeature.Benchmarks" />
23+
<InternalsVisibleTo Include="OpenFeature.Tests" />
24+
<InternalsVisibleTo Include="OpenFeature.E2ETests" />
25+
<InternalsVisibleTo Include="OpenFeature.DependencyInjection" />
26+
<InternalsVisibleTo Include="OpenFeature.Providers.MultiProvider" />
27+
<InternalsVisibleTo Include="OpenFeature.Providers.MultiProvider.Tests" />
28+
<None Include="../../README.md" Pack="true" PackagePath="/" />
29+
</ItemGroup>
2730

28-
</Project>
31+
</Project>

0 commit comments

Comments
 (0)