Skip to content

Switch version from function to string in zpages#8325

Merged
dmathieu merged 9 commits into
open-telemetry:mainfrom
sonalgaud12:zpages-vchange
Jan 5, 2026
Merged

Switch version from function to string in zpages#8325
dmathieu merged 9 commits into
open-telemetry:mainfrom
sonalgaud12:zpages-vchange

Conversation

@sonalgaud12
Copy link
Copy Markdown
Contributor

Change Version() func to const Version string in zpages

Part of #8272

@sonalgaud12 sonalgaud12 requested review from a team and dashpole as code owners December 21, 2025 07:07
Comment thread CHANGELOG.md Outdated
Comment thread zpages/version.go Outdated
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.3%. Comparing base (c935768) to head (8017625).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #8325     +/-   ##
=======================================
- Coverage   82.3%   82.3%   -0.1%     
=======================================
  Files        193     192      -1     
  Lines      13769   13767      -2     
=======================================
- Hits       11334   11332      -2     
  Misses      2030    2030             
  Partials     405     405             
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sonalgaud12 sonalgaud12 requested a review from MrAlias December 23, 2025 15:07
Comment thread CHANGELOG.md Outdated
@sonalgaud12 sonalgaud12 requested a review from MrAlias January 4, 2026 08:58
@dmathieu dmathieu merged commit 1672ef9 into open-telemetry:main Jan 5, 2026
40 of 45 checks passed
@MrAlias MrAlias added this to the v1.40.0 milestone Jan 30, 2026
@sonalgaud12 sonalgaud12 deleted the zpages-vchange branch March 12, 2026 17:16
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.

5 participants