Conversation
Agent-Logs-Url: https://github.com/shibayan/CsvHelper.FastDynamic/sessions/7b906364-7d2a-4656-94a2-00f03d4c683a Co-authored-by: shibayan <1356444+shibayan@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
リポジトリ内の .NET ターゲットフレームワークを net8.0 に統一し、直接参照している NuGet 依存(CsvHelper / xUnit / BenchmarkDotNet 等)と CI の .NET SDK バージョンを最新化する PR です。
Changes:
- 各プロジェクトの
TargetFrameworkをnet8.0に統一 - 主要 NuGet パッケージ(CsvHelper / テスト関連 / BenchmarkDotNet)を更新
- GitHub Actions の
DOTNET_VERSIONを8.0.xに更新
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| CsvHelper.FastDynamic/CsvHelper.FastDynamic.csproj | ライブラリの TFM を net8.0 に更新し、CsvHelper を更新 |
| CsvHelper.FastDynamic.Tests/CsvHelper.FastDynamic.Tests.csproj | テストプロジェクトの TFM とテスト依存(Test SDK/xUnit)を更新 |
| CsvHelper.FastDynamic.Performance/CsvHelper.FastDynamic.Performance.csproj | ベンチマーク用プロジェクトの TFM と BenchmarkDotNet を更新 |
| .github/workflows/publish.yml | publish ワークフローの .NET SDK バージョンを 8.0.x へ更新 |
| .github/workflows/build.yml | build/test/lint ワークフローの .NET SDK バージョンを 8.0.x へ更新 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| <PropertyGroup> | ||
| <TargetFramework>net6.0</TargetFramework> | ||
| <TargetFramework>net8.0</TargetFramework> |
There was a problem hiding this comment.
ライブラリプロジェクトを単一ターゲットの net8.0 にすると、net6/net7 を含む旧TFMのアプリから参照できなくなるため NuGet 利用者に対して破壊的変更になります。互換性を維持したい場合は multi-target(例: net6.0;net8.0 など)にするか、破壊的変更としてリリース方針(メジャーバージョン更新/リリースノート)を明確にしてください。
| <TargetFramework>net8.0</TargetFramework> | |
| <TargetFrameworks>net6.0;net8.0</TargetFrameworks> |
| env: | ||
| DOTNET_VERSION: 7.0.x | ||
| DOTNET_VERSION: 8.0.x | ||
|
|
There was a problem hiding this comment.
DOTNET_VERSION を 8.0.x に上げるタイミングで、actions/checkout と actions/setup-dotnet も v4 へ更新しておくのが安全です(v3 系は古い Node ランタイム依存のため、ランナー側の更新で将来的に動かなくなるリスクがあります)。
| env: | ||
| DOTNET_VERSION: 7.0.x | ||
| DOTNET_VERSION: 8.0.x | ||
|
|
There was a problem hiding this comment.
DOTNET_VERSION を 8.0.x に上げるタイミングで、actions/checkout と actions/setup-dotnet も v4 へ更新しておくのが安全です(v3 系は古い Node ランタイム依存のため、ランナー側の更新で将来的に動かなくなるリスクがあります)。
リポジトリ全体のターゲットフレームワークが
net6.0/net7.0に分散しており、サポート期限と依存関係の陳腐化が進んでいました。全プロジェクトをnet8.0に統一し、直接参照している NuGet パッケージを最新安定版へ更新しています。Framework baseline の統一
CsvHelper.FastDynamic,CsvHelper.FastDynamic.Tests,CsvHelper.FastDynamic.PerformanceのTargetFrameworkをすべてnet8.0に更新。NuGet 依存の最新化(direct dependencies)
CsvHelper30.0.1→33.1.0Microsoft.NET.Test.Sdk17.7.2→18.4.0xunit2.5.0→2.9.3xunit.runner.visualstudio2.5.0→3.1.5BenchmarkDotNet0.13.7→0.15.8CI/CD 実行環境の追従
DOTNET_VERSIONを7.0.xから8.0.xに更新(build/publish workflow)。