Skip to content

Capitalize env var to avoid case-sensitive conflicts#137

Merged
hectorhdzg merged 3 commits into
microsoft:mainfrom
rads-1996:fix-env-var
May 12, 2026
Merged

Capitalize env var to avoid case-sensitive conflicts#137
hectorhdzg merged 3 commits into
microsoft:mainfrom
rads-1996:fix-env-var

Conversation

@rads-1996
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings May 12, 2026 18:27
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the distro’s internal constant for the “version propagation” environment variable to use an all-caps name, aligning it with typical env-var naming and reducing case-related ambiguity.

Changes:

  • Renamed the version propagation env var from microsoft_opentelemetry_version to MICROSOFT_OPENTELEMETRY_VERSION.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/microsoft/opentelemetry/_constants.py
@hectorhdzg hectorhdzg merged commit 3714dc6 into microsoft:main May 12, 2026
8 checks passed
@rads-1996 rads-1996 deleted the fix-env-var branch May 12, 2026 21:20
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.

4 participants