Skip to content

Migrate chapter 4 main solution to .NET 10 and add central package management#238

Merged
meaboutsoftware merged 5 commits intomainfrom
feature/chapter-4-add-central-package-mgmt
Jan 25, 2026
Merged

Migrate chapter 4 main solution to .NET 10 and add central package management#238
meaboutsoftware merged 5 commits intomainfrom
feature/chapter-4-add-central-package-mgmt

Conversation

@meaboutsoftware
Copy link
Copy Markdown
Collaborator

@meaboutsoftware meaboutsoftware commented Jan 25, 2026

📋 Description

This PR migrates Chapter 4 main solution (Fitnet) to .NET 10, introduces Central Package Management for better dependency consistency, and fixes GitHub Actions workflow authentication issues with the GitHub Packages NuGet feed.

Key Changes:

  • Migrated all projects from .NET 9 to .NET 10
  • Implemented Central Package Management using Directory.Packages.props
  • Fixed workflow authentication by switching from add source to update source with local config file
  • Updated all NuGet packages to their latest .NET 10 compatible versions

Issue Resolved: Build pipeline was failing with "The name specified has already been added to the list of available package sources" due to attempting to add a source that already existed in local nuget.config.

📦 PR Includes

  • Feature added 🆕 (Central Package Management)
  • Bug fix 🐛 (GitHub Actions workflow authentication)
  • Code refactor 🔄 (.NET 10 migration, package consolidation)
  • Documentation update 📚
  • Tests added/updated 🧪
  • Other: Infrastructure upgrade to .NET 10

🚫 PR Does Not Include

  • Breaking changes ❌
  • Major architectural changes 🏗️
  • Unrelated features/tasks 🚫

💡 Additional Notes

Central Package Management Benefits:

  • Single source of truth for all package versions in Directory.Packages.props
  • Eliminates version conflicts across projects
  • Simplifies dependency updates and maintenance
  • Aligns with .NET best practices and modern solution structure

Workflow Fix Details:

  • Changed authentication approach from adding new source to updating existing source
  • Added --configfile nuget.config to work with local configuration instead of global
  • Fixed variable references: now using ${{ github.repository_owner }} and ${{ secrets.GITHUB_TOKEN }}
  • Corrected source name casing to EvolutionaryArchitecture (matching nuget.config)

Testing:

  • ✅ Solution builds successfully locally and in CI/CD
  • ✅ All existing tests pass
  • ✅ Package restore works with authenticated GitHub Packages feed
  • ✅ GitHub Actions workflow completes without errors

@meaboutsoftware meaboutsoftware force-pushed the feature/chapter-4-add-central-package-mgmt branch from d3a28f8 to c323493 Compare January 25, 2026 08:28
@meaboutsoftware meaboutsoftware force-pushed the feature/chapter-4-add-central-package-mgmt branch from c323493 to 08bb10c Compare January 25, 2026 08:30
@meaboutsoftware meaboutsoftware merged commit 01ca766 into main Jan 25, 2026
2 checks passed
@meaboutsoftware meaboutsoftware deleted the feature/chapter-4-add-central-package-mgmt branch January 25, 2026 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants