Phase 1: Foundation and Preparation for .NET 8 Migration #14
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Phase 1: Foundation and Preparation for .NET 8 Migration
Summary
This PR prepares the SampleMvcWebApp for eventual .NET 8 migration by upgrading the Autofac dependency injection framework and creating comprehensive documentation. All changes maintain compatibility with .NET Framework 4.5.1.
Package Upgrades:
Documentation Added:
docs/Phase1-DependencyInventory.md- Full dependency analysis with .NET 8 compatibility statusdocs/Phase1-LoggingMigrationPlan.md- Strategy for log4net → Microsoft.Extensions.Loggingdocs/Phase1-PerformanceBaseline.md- Performance metrics baseline for post-migration comparisonTests Added:
Test12AutofacUpgradeTests.cs- 14 unit tests verifying DI functionality with upgraded AutofacReview & Testing Checklist for Human
Test12AutofacUpgradeTests.cscompiles but couldn't be run due to LocalDB dependencies; verify all 14 tests pass on WindowsRecommended Test Plan:
Notes
AutofacDi.csorServiceLayerModule.cs- the Autofac 4.x API appears backward compatibleLink to Devin run: https://app.devin.ai/sessions/b2c468c87cc446489fe1be94f310c42f
Requested by: @mbatchelor81