Skip to content

Mark Golang hosting package as obsolete#1323

Open
DavidGarton8 wants to merge 2 commits into
CommunityToolkit:mainfrom
DavidGarton8:golang_obsolete
Open

Mark Golang hosting package as obsolete#1323
DavidGarton8 wants to merge 2 commits into
CommunityToolkit:mainfrom
DavidGarton8:golang_obsolete

Conversation

@DavidGarton8
Copy link
Copy Markdown
Contributor

Closes #1315

Marks the CommunityToolkit.Aspire.Hosting.Golang package as obsolete in favor of Aspire.Hosting.Go.

This change:

  • Adds Obsolete attributes to the legacy extension type, public extension methods, and legacy resource types
  • Updates obsolete messages to direct users to Aspire.Hosting.Go
  • Updates the package README to state that this package is obsolete and that new applications should use Aspire.Hosting.Go
  • Suppresses CS0618 warnings in test and example projects that intentionally continue using the legacy APIs

No functional behavior was changed.

PR Checklist

  • Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
  • Based off latest main branch of toolkit
  • PR doesn't include merge commits (always rebase on top of our main, if needed)
  • New integration
    • Docs are written
    • Added description of major feature to project description for NuGet package (4000 total character limit, so don't push entire description over that)
  • Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Contains NO breaking changes
  • Every new API (including internal ones) has full XML docs
  • Code follows all style conventions

Other information

This PR intentionally does not introduce a replacement implementation in this package. It only marks the existing package surface as obsolete and updates documentation to direct users to Aspire.Hosting.Go.

This PR should be merged when 13.4.0 is released, so the obsoletion guidance can direct users to Aspire.Hosting.Go once that package is publicly available.

The `CommunityToolkit.Aspire.Hosting.Golang` package and its members have been marked as obsolete in favor of the newer `Aspire.Hosting.Go` package. Added warnings and updated documentation to guide users toward migrating to the replacement package. No functional changes were made, and the old package remains available for compatibility.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 16, 2026

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.sh | bash -s -- 1323

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.ps1) } 1323"

Copy link
Copy Markdown
Member

@aaronpowell aaronpowell left a comment

Choose a reason for hiding this comment

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

Can you remove the test project from here. Since we're disabling everything we might as well remove the tests.

@tommasodotNET
Copy link
Copy Markdown
Contributor

Can you remove the test project from here. Since we're disabling everything we might as well remove the tests.

and the samples too, probably?

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.

Deprecate Golang integration now that Go support is first-class in Aspire

3 participants