Merge main to preview #704
Merged
Conversation
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
* add copilot instructions * fix grammar * Update .github/code-gen-instructions.md Co-authored-by: Jimmy Campbell <[email protected]> --------- Co-authored-by: Jimmy Campbell <[email protected]>
* initial fixes to jquery for codeql alerts * add other wwwroot files * remove comment * redo line endings on configstoredemo sample files * update site css to fit title * fix title spacing * update to aspnet ajax instead
Add CodeQL exclusions for code samples
* add testcase for refresh activity * update * update * update
…bility logic (#676) * add environment variable, unit test * remove comments * Update src/Microsoft.Extensions.Configuration.AzureAppConfiguration/FeatureManagement/FeatureManagementKeyValueAdapter.cs Co-authored-by: Copilot <[email protected]> * fix unit test * update comments * move bool parsing logic outside try * fix formatting * add environment variable class * update * update --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: Zhiyuan Liang <[email protected]> Co-authored-by: Zhiyuan Liang <[email protected]>
Fix activity source test bug
* update endpoint in do while * add test * in progress * update test, update logic to backoff using correct endpoint * make test more specific --------- Co-authored-by: AMER JUSUPOVIC <[email protected]> Co-authored-by: Amer Jusupovic <[email protected]>
* support jsonc * remove unused reference * use private static option
* add request tracing for aspire component * update tag name
* feat: defined snapshot reference content type and Json Property * added snapshot reference content type to content type extensions * updated the snapshot reference content type to include the charset * Implementing snapshot references * Fixed the behavior of an edge case and moved file to a better location in repo * created unit tests for Snapshot References * Removed unnecessary comments from tests * removed extra comments and cleaned up code * modified test and added set up for testing snapshot references * adding more integration tests for snapshot references * removed client and cancellation token from snapshot reference class and updated all affected code * updated namespaces for all files and created parser class * additional file to update namespace * Modified the code to directly use the content type instead of copying * Used object initializer pattern and passed in cancellationTokens to async methods * fixed comment and updated TestContext format * updated behavior to throw exception if snapshot name is null * Moved the exception error messages from inline to ErrorMessages class and renamed parsing method to Parse() * Added Request tracing and case for snapshot reference is registered for refresh by not called in select * added test case to test adding snapshot reference to register but not part of select * updating request tracing to only tracking use of snapshot references * removing request tracing logic for Snapshot References count * added comments and returning SnapshotReference type from Parse() * removed second way of checking for snapshot references type and updated test cases * removed redundant error message * removed unnecessary code * fixed whitespace issues and made error message clearer * updating naming and removing old telemetry code * updated namespace and directory to SnapshotReference * Added new JsonFields type * fixed error regarding same namespace and type * Updated the Parse logic to handle all exceptions instead of LoadSnapshotData * updated comments to method * removed the update and reset snapshot reference request tracing methods as they were unnecessary * removed requestTracing for SnapshotReference from refresh * reverting previous change * Made these keyvault integration tests more resilient * Cleaning up code * More nit changes * updated request tracing for snapshot references in refresh * making the key vault integration test more resilient * correction to which test needed resilience
Drop .net 6 & Upgrade dependency
* delete kv sequentially * fix snapshot creation and deletion bug * fix * update retention period
…694) * add overload to take endpoint and credential * enrich comment for intellisense * update comment to point to use options.connect pattern * update
Add AlwaysHealthy health check when provider is disabled
RichardChen820
approved these changes
Oct 10, 2025
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.
No description provided.