Skip to content

Conversation

@devin-ai-integration
Copy link

Phase 1: Convert project files to SDK-style format and migrate to .NET 6

Summary

This PR implements Phase 1 of the .NET Framework 4.5.1 to .NET Core 6 migration by converting all project files to modern SDK-style format and migrating package management from packages.config to PackageReference format.

Key Changes:

  • Converted all 4 .csproj files from old-style to SDK-style format (SampleWebApp, ServiceLayer, DataLayer, Tests)
  • Updated target framework from v4.5.1 to net6.0 across all projects
  • Migrated package references from packages.config to PackageReference format
  • Removed all packages.config files (4 total)
  • Added compatibility properties to prevent build issues (GenerateAssemblyInfo=false, EnableDefaultContentItems=false)
  • Added compatibility packages (System.Configuration.ConfigurationManager, Microsoft.Windows.Compatibility)

Current Status: ⚠️ The solution does not currently build - there are 4 compilation errors related to missing System.Web types (HttpApplication, RouteCollection) that don't exist in .NET 6. This is expected for Phase 1 as code changes are planned for subsequent phases.

Review & Testing Checklist for Human

  • Verify project file structure conversion - Compare old vs new .csproj format for accuracy
  • Review package reference migration - Confirm all packages from packages.config were properly migrated to PackageReference format
  • Confirm compilation failures are acceptable - The 4 System.Web-related errors are expected and will be addressed in Phase 2
  • Validate package compatibility warnings - Review the 230+ NU1701 warnings about .NET Framework packages being used with .NET 6

Test Plan: Due to compilation failures, functional testing is not possible at this stage. The next phase should focus on resolving the System.Web dependency issues and updating code to use ASP.NET Core equivalents.

Notes

This is Phase 1 of a multi-phase migration plan. The project structure has been successfully modernized, but code changes will be required in Phase 2 to replace ASP.NET Framework-specific types with ASP.NET Core equivalents.

Link to Devin run: https://app.devin.ai/sessions/6bb3e4352c164fa4bacab472aa211b7b
Requested by: ben-windsurf (@ben-windsurf)

- Convert all four .csproj files from old-style to SDK-style format
- Update target framework from v4.5.1 to net6.0
- Migrate NuGet package references from packages.config to PackageReference format
- Remove all packages.config files
- Add compatibility packages for .NET Framework dependencies
- Add GenerateAssemblyInfo=false to prevent duplicate assembly attributes
- Add EnableDefaultContentItems=false for SampleWebApp to prevent duplicate content items

Note: Build currently fails due to missing System.Web types (HttpApplication, RouteCollection)
that require code changes beyond Phase 1 scope. These will be addressed in subsequent phases.

Co-Authored-By: ben.lehrburger@windsurf.com <ben.lehrburger@windsurf.com>
@devin-ai-integration
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

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