Open
Conversation
Signed-off-by: André Silva <[email protected]>
…re.Providers.Flagd Signed-off-by: André Silva <[email protected]>
Contributor
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.sh | bash -s -- 1272Or
iex "& { $(irm https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.ps1) } 1272" |
Contributor
There was a problem hiding this comment.
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.Flagdand bumps the centrally-managed version to0.6.0. - Updates affected
.csprojpackage references to the new package ID. - Updates
usingdirectives 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. |
Co-authored-by: Copilot <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Signed-off-by: André Silva [email protected]
This pull request updates the codebase to use the new
OpenFeature.Providers.Flagdpackage instead of the deprecatedOpenFeature.Contrib.Providers.Flagdpackage. The changes ensure that all references, package dependencies, and using statements are consistent with the new package version.Dependency and Reference Updates:
Directory.Packages.propsto useOpenFeature.Providers.Flagdversion 0.6.0 instead ofOpenFeature.Contrib.Providers.Flagd..csproj) to referenceOpenFeature.Providers.Flagdinstead ofOpenFeature.Contrib.Providers.Flagd. [1] [2]Code Changes:
usingstatements in code and test files to referenceOpenFeature.Providers.Flagdinstead ofOpenFeature.Contrib.Providers.Flagd. [1] [2]PR Checklist
Other information
This only affects unit tests. Integration is not affected.