Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 18 additions & 18 deletions src/BootstrapBlazor/Directory.Build.props
Original file line number Diff line number Diff line change
Expand Up @@ -32,42 +32,42 @@
<ItemGroup Condition="'$(TargetFramework)' == 'net6.0'">
<PackageReference Include="Microsoft.AspNetCore.Components.Authorization" Version="$(NET6Version)" />
<PackageReference Include="Microsoft.AspNetCore.Components.Web" Version="$(NET6Version)" />
<PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="$(NET6Version)" />
<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="$(NET6Version)" />
<PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Localization" Version="$(NET6Version)" />
<PackageReference Include="Microsoft.Extensions.Http" Version="$(NET6Version)" />
<PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" Version="$(NET6Version)" />
<PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="$(NET6Version)" />
<PackageReference Include="Microsoft.Extensions.Http" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="$(NET8Version)" />
Copy link

Copilot AI Jan 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upgrading Microsoft.Extensions packages from NET6Version to NET8Version for a net6.0 target framework creates an inconsistency. While these packages may technically work due to backward compatibility, it's better practice to keep the package versions aligned with the target framework version unless there's a specific reason for the upgrade. This ensures consistency and avoids potential compatibility issues.

Suggested change
<PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="$(NET6Version)" />

Copilot uses AI. Check for mistakes.
</ItemGroup>

<ItemGroup Condition="'$(TargetFramework)' == 'net7.0'">
<PackageReference Include="Microsoft.AspNetCore.Components.Authorization" Version="$(NET7Version)" />
<PackageReference Include="Microsoft.AspNetCore.Components.Web" Version="$(NET7Version)" />
<PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="$(NET7Version)" />
<PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Localization" Version="$(NET7Version)" />
<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="$(NET7Version)" />
<PackageReference Include="Microsoft.Extensions.Http" Version="$(NET7Version)" />
<PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" Version="$(NET7Version)" />
<PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="$(NET7Version)" />
<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Http" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="$(NET8Version)" />
Comment on lines +46 to +51
Copy link

Copilot AI Jan 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upgrading Microsoft.Extensions packages from NET7Version to NET8Version for a net7.0 target framework creates an inconsistency. While these packages may work due to backward compatibility, it's better practice to keep the package versions aligned with the target framework version unless there's a specific reason for the upgrade. This ensures consistency and avoids potential compatibility issues.

Copilot uses AI. Check for mistakes.
</ItemGroup>

<ItemGroup Condition="'$(TargetFramework)' == 'net8.0'">
<PackageReference Include="Microsoft.AspNetCore.Components.Authorization" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.AspNetCore.Components.Web" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Http" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" Version="$(NET8Version)" />
<PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="$(NET10Version)" />
<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="$(NET10Version)" />
<PackageReference Include="Microsoft.Extensions.Http" Version="$(NET10Version)" />
<PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" Version="$(NET10Version)" />
<PackageReference Include="Microsoft.Extensions.Localization" Version="$(NET8Version)" />
</ItemGroup>

<ItemGroup Condition="'$(TargetFramework)' == 'net9.0'">
<PackageReference Include="Microsoft.AspNetCore.Components.Authorization" Version="$(NET9Version)" />
<PackageReference Include="Microsoft.AspNetCore.Components.Web" Version="$(NET9Version)" />
<PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="$(NET9Version)" />
<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="$(NET9Version)" />
<PackageReference Include="Microsoft.Extensions.Http" Version="$(NET9Version)" />
<PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" Version="$(NET9Version)" />
<PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="$(NET10Version)" />
<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="$(NET10Version)" />
<PackageReference Include="Microsoft.Extensions.Http" Version="$(NET10Version)" />
<PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" Version="$(NET10Version)" />
<PackageReference Include="Microsoft.Extensions.Localization" Version="$(NET9Version)" />
</ItemGroup>

Expand Down