Skip to content

Conversation

@geraldbarre-workleap
Copy link
Contributor

@geraldbarre-workleap geraldbarre-workleap commented Nov 20, 2025

Jira issue link: feng-1466

Description of changes

Breaking changes

Additional checks

  • Updated the documentation of the project to reflect the changes
  • Added new tests that cover the code changes

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 updates .NET Aspire from version 9.5.2 to 13.0.0, which includes a breaking API change requiring migration from the lifecycle hook pattern to an eventing-based pattern. The update also includes related dependency upgrades.

  • Migration of all lifecycle hooks from IDistributedApplicationLifecycleHook to IDistributedApplicationEventingSubscriber with the new eventing pattern
  • Update of Microsoft.Extensions.* packages from 9.0.11 to 10.0.0
  • Update of various other dependencies including Spectre.Console, NuGet.Protocol, and Microsoft.Identity.Client packages
  • Minor code refactoring to simplify null-checking pattern

Reviewed Changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/Leap.Cli/Leap.Cli.csproj Updates Aspire SDK and packages from 9.5.2 to 13.0.0, Microsoft.Extensions.* from 9.0.11 to 10.0.0, and other dependency version bumps
src/Workleap.Extensions.Leap/Workleap.Extensions.Leap.csproj Updates Microsoft.Extensions packages from 9.0.11 to 10.0.0
src/Leap.Testing/Leap.Testing.csproj Updates Microsoft.Extensions.Http.Polly from 9.0.11 to 10.0.0
src/Leap.Cli.Tests/Leap.Cli.Tests.csproj Updates Spectre.Console.Testing from 0.53.0 to 0.54.0
src/Leap.Cli.Tests/LeapTests.cs Adds blank line after namespace declaration (formatting)
src/Leap.Cli/Platform/Telemetry/TelemetryHelper.cs Refactors null check to use null-conditional operator for cleaner code
src/Leap.Cli/Pipeline/StartAzureCliDockerProxyPipelineStep.cs Migrates lifecycle hook to new eventing subscriber pattern with BeforeStartEvent
src/Leap.Cli/Pipeline/PrepareReverseProxyPipelineStep.cs Migrates lifecycle hook to new eventing subscriber pattern with BeforeStartEvent
src/Leap.Cli/Aspire/UseLeapCertificateForAspireDashboardLifecycleHook.cs Migrates lifecycle hook to new eventing subscriber pattern with BeforeStartEvent
src/Leap.Cli/Aspire/ReverseProxyEndpointExtensions.cs Migrates lifecycle hook to new eventing subscriber pattern with BeforeStartEvent
src/Leap.Cli/Aspire/EnableAspireDashboardDiagnosticLoggingLifecycleHook.cs Migrates lifecycle hook to new eventing subscriber pattern with BeforeStartEvent
src/Leap.Cli/Aspire/DotnetExecutableResource.cs Migrates lifecycle hook to new eventing subscriber pattern with BeforeStartEvent
src/Leap.Cli/Aspire/DockerComposeResourceLifecycleHook.cs Migrates lifecycle hook to new eventing subscriber pattern with BeforeStartEvent and reorganizes imports
src/Leap.Cli/Aspire/DockerComposeResourceExtensions.cs Updates registration from TryAddLifecycleHook to TryAddEventingSubscriber
src/Leap.Cli/Aspire/DistributedApplicationBuilderExtensions.cs Updates registration from TryAddLifecycleHook to TryAddEventingSubscriber
src/Leap.Cli/Aspire/DetectDotnetBuildRaceConditionErrorLifecycleHook.cs Migrates lifecycle hook to new eventing subscriber pattern with BeforeStartEvent
src/Leap.Cli/Aspire/AspireManager.cs Updates registrations from TryAddLifecycleHook to TryAddEventingSubscriber and IDistributedApplicationLifecycleHook to IDistributedApplicationEventingSubscriber
src/Leap.Cli/Aspire/AspireDashboardReadinessAwaiter.cs Migrates lifecycle hook to new eventing subscriber pattern with BeforeStartEvent

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

@geraldbarre-workleap geraldbarre-workleap merged commit da374d9 into main Nov 20, 2025
21 checks passed
@geraldbarre-workleap geraldbarre-workleap deleted the feature/feng-1466 branch November 20, 2025 21:14
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.

3 participants