Skip to content

Merge main into preview #564

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Jun 21, 2024
Merged

Merge main into preview #564

merged 11 commits into from
Jun 21, 2024

Conversation

amerjusupovic
Copy link
Member

No description provided.

amerjusupovic and others added 11 commits April 3, 2024 13:49
* target .NET 8 and update other testing packages

* use version of dependency injection equal to net version

* remove test comment

* fix json content test

* revert change to flattener

* change back version of json package

* add back versions

* fix test to work for new versions of microsoft.extensions.configuration.json

* remove conditional packages

* fix spacing, test with ado pipeline

* fix spacing

* update other examples

* allow informationalversionattribute format in test for useragentheader
Bumps [Azure.Identity](https://github.com/Azure/azure-sdk-for-net) from 1.10.2 to 1.11.0.
- [Release notes](https://github.com/Azure/azure-sdk-for-net/releases)
- [Commits](Azure/azure-sdk-for-net@Azure.Identity_1.10.2...Azure.Identity_1.11.0)

---
updated-dependencies:
- dependency-name: Azure.Identity
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [Azure.Identity](https://github.com/Azure/azure-sdk-for-net) from 1.10.2 to 1.11.0.
- [Release notes](https://github.com/Azure/azure-sdk-for-net/releases)
- [Commits](Azure/azure-sdk-for-net@Azure.Identity_1.10.2...Azure.Identity_1.11.0)

---
updated-dependencies:
- dependency-name: Azure.Identity
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Amer Jusupovic <32405726+amerjusupovic@users.noreply.github.com>
* WIP adding utf8jsonreader

* WIP jsondocument

* replace all json adapters with jsondocument

* combine code to serialize feature flag with code for adding key values to configuration

* remove unused classes from jsonserializer approach

* remove jsonserializer from testhelpers

* WIP add handling and error messages for invalid values

* WIP adding error messages

* use method to create formatexception

* validate for key vault uri and throw exception, fix logic in featuremanagement adapter

* update error message, fix expected type for enabled

* WIP

* WIP testing aot compatible

* small logic updates

* improve variable names

* edit error message

* add isaotcompatible property to packages

* fix error message for bool

* WIP use utf8jsonreader

* WIP

* use utf8jsonreader for feature flags

* use utf8jsonreader for keyvault secret reference

* PR comment revision

* WIP adding tests, PR revisions

* WIP add some tests, PR revisions

* key vault tests, use keyvaultreferenceexception for all scenarios because it's the established pattern

* fix missing setup

* PR revisions

* update tests again, PR revisions

* don't handle invalidoperationexception

* remove unused exception var
…ity in example projects (#560)

* update package

* update package
…551)

* add log for formatexception from invalid feature flag json

* remove capitalized json

* remove unused using

* fix comments

* fix constant reference
@amerjusupovic amerjusupovic requested a review from avanigupta June 21, 2024 21:40
@amerjusupovic amerjusupovic merged commit 1a66edb into preview Jun 21, 2024
2 checks passed
@amerjusupovic amerjusupovic deleted the merge-main-into-preview branch June 21, 2024 23:08
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.

3 participants