Skip to content

Conversation

@kamilbaczek
Copy link
Collaborator

@kamilbaczek kamilbaczek commented Nov 14, 2025

Changes:

  • Replaced Docker Compose with .NET Aspire

Rationale:

  • .NET Aspire enhances local development, making projects seamless to run
  • Strong typing with .NET reduces potential for errors compared to Docker Compose scripts
  • Adds observability UI and simplifies connection string management

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates the local development environment from Docker Compose to .NET Aspire, introducing a new Fitnet.AppHost project for orchestration. The change aims to improve developer experience through strong typing, built-in observability, and simplified connection string management.

Key Changes:

  • Added Fitnet.AppHost project with .NET Aspire 9.0 for orchestrating PostgreSQL and the Fitnet application
  • Removed Docker Compose configuration (docker-compose.yml, Dockerfile, .dockerignore)
  • Updated SonarAnalyzer.CSharp package to version 10.15.0.120848 across all test projects
  • Modified application URL ports in launchSettings.json to avoid conflicts

Reviewed changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
README.adoc Added .NET Aspire to the technology stack list
Chapter-1-initial-architecture/Src/docker-compose.yml Removed Docker Compose configuration in favor of Aspire
Chapter-1-initial-architecture/Src/Dockerfile Removed Dockerfile as containerization is now handled by Aspire
Chapter-1-initial-architecture/Src/Fitnet/.dockerignore Removed Docker ignore file (no longer needed)
Chapter-1-initial-architecture/Src/Fitnet/Properties/launchSettings.json Updated ports to avoid conflicts (7097→7098, 5013→5014)
Chapter-1-initial-architecture/Src/Fitnet/Program.cs Removed module registration comment
Chapter-1-initial-architecture/Src/Fitnet/Fitnet.csproj Removed docker-compose.yml and Dockerfile content links
Chapter-1-initial-architecture/Src/Fitnet.AppHost/* Added new Aspire AppHost project with PostgreSQL and Fitnet application orchestration
Chapter-1-initial-architecture/Src/Fitnet.slnx Added Fitnet.AppHost project and updated project type attributes
Chapter-1-initial-architecture/Src/.UnitTests/.csproj Updated SonarAnalyzer.CSharp to version 10.15.0.120848
Chapter-1-initial-architecture/Src/.IntegrationTests/.csproj Updated SonarAnalyzer.CSharp to version 10.15.0.120848
Chapter-1-initial-architecture/Src/.ArchitectureTests/.csproj Updated SonarAnalyzer.CSharp to version 10.15.0.120848
Chapter-1-initial-architecture/Src/Fitnet//Migrations/.cs Added blank lines after namespace declarations for consistent formatting
Chapter-1-initial-architecture/README.adoc Updated local development instructions for Aspire

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kamilbaczek kamilbaczek merged commit b8602fd into main Dec 7, 2025
2 checks passed
@kamilbaczek kamilbaczek deleted the feature/chapter-1-dotnet-aspire branch December 7, 2025 09:27
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.

4 participants