Skip to content

Conversation

joperezr
Copy link
Member

cc: @DamianEdwards @davidfowl @maddymontaquila

FYI: @marcpopMSFT we are planning to ship one last version of the aspire workload that adds a warning for users still relying on it to build projects (meaning they depend on Aspire 8.x). The intent is to notify them so they can move to a newer and supported version of aspire.

Description

Add Build warning to projects depending on the workload notifying them that aspire workload is deprecated.

Checklist

… notifying them that aspire workload is deprecated.
@Copilot Copilot AI review requested due to automatic review settings July 18, 2025 20:57
Copy link
Contributor

@Copilot 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 adds a build warning/error to notify users that the Aspire workload is deprecated and they should migrate to using Aspire as NuGet packages instead. This is part of the 8.2.3 release to inform users still on older versions.

  • Adds a new MSBuild target that triggers an error (ASPIRE008) for projects using the deprecated workload
  • Increments the patch version from 8.2.2 to 8.2.3 for this deprecation notice release
  • Suppresses the new error for the repo itself since it still uses the workload version internally

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
src/Aspire.Hosting.Sdk/SDK/Sdk.in.targets Adds new build target with deprecation error message
eng/Versions.props Bumps patch version to 8.2.3 for this release
Directory.Build.props Suppresses the deprecation error for the repo itself with explanatory comment

joperezr and others added 3 commits July 18, 2025 16:18
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@joperezr joperezr requested a review from radical as a code owner July 18, 2025 23:45
@joperezr
Copy link
Member Author

Closing as after some discussion this should instead go in the SDK.

@joperezr joperezr closed this Jul 21, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Aug 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant