Skip to content

Remove netcoreapp3.1 from source TargetFrameworks#113

Merged
Chris-Wolfgang merged 1 commit into
mainfrom
fix/remove-netcoreapp31-from-source
Mar 28, 2026
Merged

Remove netcoreapp3.1 from source TargetFrameworks#113
Chris-Wolfgang merged 1 commit into
mainfrom
fix/remove-netcoreapp31-from-source

Conversation

@Chris-Wolfgang
Copy link
Copy Markdown
Owner

Summary

  • Remove netcoreapp3.1 from source project TargetFrameworks
  • Keep netcoreapp3.1 in test project — exercises netstandard2.0 code paths
  • Remove netcoreapp3.1 from package reference condition (no longer a source TFM)

Test plan

  • Build clean on all remaining source TFMs
  • netcoreapp3.1 tests still pass (running against netstandard2.0 build)
  • Coverage unchanged

🤖 Generated with Claude Code

@Chris-Wolfgang Chris-Wolfgang force-pushed the fix/remove-netcoreapp31-from-source branch from 145ff35 to be7bd58 Compare March 27, 2026 23:04
@Chris-Wolfgang Chris-Wolfgang force-pushed the fix/remove-netcoreapp31-from-source branch 2 times, most recently from 4c73149 to 99b4d6d Compare March 28, 2026 12:56
With netcoreapp3.1 removed from source, the netcoreapp3.1 tests
now run against the netstandard2.0 assembly, providing direct
coverage of those code paths. The runsettings file and the
ExcludeFromCodeCoverage property are no longer needed.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant