Skip to content

Swap deprecated flagd package#1272

Open
askpt wants to merge 3 commits intoCommunityToolkit:mainfrom
askpt:askpt/swap-flagd
Open

Swap deprecated flagd package#1272
askpt wants to merge 3 commits intoCommunityToolkit:mainfrom
askpt:askpt/swap-flagd

Conversation

@askpt
Copy link
Copy Markdown
Contributor

@askpt askpt commented Apr 10, 2026

Signed-off-by: André Silva [email protected]

This pull request updates the codebase to use the new OpenFeature.Providers.Flagd package instead of the deprecated OpenFeature.Contrib.Providers.Flagd package. The changes ensure that all references, package dependencies, and using statements are consistent with the new package version.

Dependency and Reference Updates:

  • Updated the package version in Directory.Packages.props to use OpenFeature.Providers.Flagd version 0.6.0 instead of OpenFeature.Contrib.Providers.Flagd.
  • Updated all project files and test projects (.csproj) to reference OpenFeature.Providers.Flagd instead of OpenFeature.Contrib.Providers.Flagd. [1] [2]

Code Changes:

  • Updated all using statements in code and test files to reference OpenFeature.Providers.Flagd instead of OpenFeature.Contrib.Providers.Flagd. [1] [2]

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)
  • 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 only affects unit tests. Integration is not affected.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 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 -- 1272

Or

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

@askpt askpt marked this pull request as ready for review April 10, 2026 18:11
Copilot AI review requested due to automatic review settings April 10, 2026 18:11
Copy link
Copy Markdown
Contributor

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

Updates the Flagd-related sample and test projects to use the non-deprecated OpenFeature.Providers.Flagd NuGet package (and namespace) instead of OpenFeature.Contrib.Providers.Flagd, keeping dependencies consistent across the repo.

Changes:

  • Replaces the deprecated Flagd provider package with OpenFeature.Providers.Flagd and bumps the centrally-managed version to 0.6.0.
  • Updates affected .csproj package references to the new package ID.
  • Updates using directives in the Flagd AppHost sample and Flagd tests to the new namespace.

Reviewed changes

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

Show a summary per file
File Description
Directory.Packages.props Swaps central package version entry to OpenFeature.Providers.Flagd 0.6.0.
tests/CommunityToolkit.Aspire.Hosting.Flagd.Tests/CommunityToolkit.Aspire.Hosting.Flagd.Tests.csproj Updates test project package reference to the new Flagd provider package ID.
tests/CommunityToolkit.Aspire.Hosting.Flagd.Tests/AppHostTests.cs Updates Flagd provider namespace import for tests.
examples/flagd/CommunityToolkit.Aspire.Hosting.Flagd.AppHost/Program.cs Updates Flagd provider namespace import in the sample AppHost.
examples/flagd/CommunityToolkit.Aspire.Hosting.Flagd.AppHost/CommunityToolkit.Aspire.Hosting.Flagd.AppHost.csproj Updates sample AppHost package reference to the new Flagd provider package ID.

Comment thread examples/flagd/CommunityToolkit.Aspire.Hosting.Flagd.AppHost/Program.cs Outdated
Comment thread examples/flagd/CommunityToolkit.Aspire.Hosting.Flagd.AppHost/Program.cs Outdated
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.

2 participants