Skip to content

App Config with Exp Telemetry #40461

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

Closed
wants to merge 17 commits into from

Conversation

mrm9084
Copy link
Member

@mrm9084 mrm9084 commented Apr 10, 2025

Description

Not intended to be merged, showing diff for beta release.

mrm9084 and others added 15 commits October 9, 2024 09:49
* Adding Telemetry

* Telemetry Support

* fixing formatting

* Update _azureappconfigurationprovider.py

* Update _azureappconfigurationproviderasync.py

* formatting

* changing doc style due to pylint-next

* fixing kwargs docs

* Formatting

* Review comments

* Changed label checking.

* black format changes

* pylint

* Update sdk/appconfiguration/azure-appconfiguration-provider/azure/appconfiguration/provider/_azureappconfigurationprovider.py

Co-authored-by: Avani Gupta <[email protected]>

* added space checks

* Update conftest.py

* moved telemetry to client wrapper

* fixing format

* updating after merge

* fixing black issue

* removing unused imports

* AllocationId

* Update CODEOWNERS

* Update CODEOWNERS

* fixing issues

* Update _client_manager_base.py

* Fixing configuration value empty in calc

* fixing pylint

* Update _constants.py

* review comments

* fixing allocation check

* format fix

---------

Co-authored-by: Avani Gupta <[email protected]>
* Update CHANGELOG.md

* Update CHANGELOG.md
* updated calc to sort keys

* Update CHANGELOG.md
* updated calc to sort keys

* Update CHANGELOG.md

* Update _client_manager_base.py
@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

@rossgrambo rossgrambo self-requested a review April 10, 2025 18:12
@mrm9084 mrm9084 closed this Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants