Repository navigation
Remove the duplicate azure monitor folder - #23
Merged
Radhika Gupta (rads-1996) merged 2 commits intoApr 9, 2026
Merged
Radhika Gupta (rads-1996) merged 2 commits into
Radhika Gupta (rads-1996) merged 2 commits into
Conversation
Radhika Gupta (rads-1996)
requested review from
Jackson Weber (JacksonWeber),
Nev (MSNev),
Hector Hernandez (hectorhdzg) and
Leighton Chen (lzchen)
as code owners
April 9, 2026 18:52
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the previously vendored/temporary azure-monitor-opentelemetry/ package folder from the repository to eliminate duplication, keeping the codebase focused on the in-repo Azure Monitor integration under src/.
Changes:
- Deleted the duplicate
azure-monitor-opentelemetry/package source tree (setup/config + Azure Monitor distro implementation files). - Updated
CHANGELOG.mdto note the removal.
Reviewed changes
Copilot reviewed 25 out of 25 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| CHANGELOG.md | Adds an entry noting removal of the duplicate Azure Monitor folder. |
| azure-monitor-opentelemetry/setup.py | Removes duplicate package build script. |
| azure-monitor-opentelemetry/README.md | Removes temporary-copy documentation. |
| azure-monitor-opentelemetry/pyproject.toml | Removes duplicate package tooling config. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/py.typed | Removes vendored typing marker file from duplicate package tree. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_version.py | Removes vendored version file from duplicate package tree. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_utils/instrumentation.py | Removes vendored instrumentation dependency-check utilities. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_utils/configurations.py | Removes vendored configuration defaulting logic. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_utils/init.py | Removes vendored diagnostics/helpers utilities. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_types.py | Removes vendored configuration typing aliases. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_diagnostics/status_logger.py | Removes vendored status logger implementation. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_diagnostics/diagnostic_logging.py | Removes vendored diagnostic logging implementation. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_diagnostics/init.py | Removes vendored diagnostics package init. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_constants.py | Removes vendored constants used by the duplicate distro. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_configure.py | Removes vendored Azure Monitor configure entrypoint implementation. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_browser_sdk_loader/snippet_injector.py | Removes vendored browser snippet injector implementation. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_browser_sdk_loader/django_middleware.py | Removes vendored Django middleware for snippet injection. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_browser_sdk_loader/_config.py | Removes vendored browser SDK loader config object. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_browser_sdk_loader/init.py | Removes vendored browser SDK loader package init/registration logic. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_autoinstrumentation/distro.py | Removes vendored OTel distro entrypoint implementation. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_autoinstrumentation/configurator.py | Removes vendored OTel configurator implementation. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/_autoinstrumentation/init.py | Removes vendored autoinstrumentation package init. |
| azure-monitor-opentelemetry/azure/monitor/opentelemetry/init.py | Removes vendored public package init exporting configure_azure_monitor. |
| azure-monitor-opentelemetry/azure/monitor/init.py | Removes vendored namespace package init. |
| azure-monitor-opentelemetry/azure/init.py | Removes vendored namespace package init. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Hector Hernandez (hectorhdzg)
approved these changes
Apr 9, 2026
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.
No description provided.