Skip to content

Fix #864 - Always serialize a brief and registry_url#929

Merged
jsuereth merged 2 commits intoopen-telemetry:mainfrom
jsuereth:fix-864
Sep 15, 2025
Merged

Fix #864 - Always serialize a brief and registry_url#929
jsuereth merged 2 commits intoopen-telemetry:mainfrom
jsuereth:fix-864

Conversation

@jsuereth
Copy link
Contributor

These are used by downstream consumers. We expect brief to always be provided going forward.

@jsuereth jsuereth requested a review from a team as a code owner September 10, 2025 17:30
@jsuereth jsuereth moved this to Next Release in OTel Weaver Project Sep 10, 2025
@codecov
Copy link

codecov bot commented Sep 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.9%. Comparing base (161a61b) to head (09a57ee).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@          Coverage Diff          @@
##            main    #929   +/-   ##
=====================================
  Coverage   77.9%   77.9%           
=====================================
  Files         76      76           
  Lines       5980    5980           
=====================================
  Hits        4660    4660           
  Misses      1320    1320           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@lquerel lquerel left a comment

Choose a reason for hiding this comment

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

LGTM

@jsuereth jsuereth enabled auto-merge (squash) September 15, 2025 12:43
@jsuereth jsuereth merged commit e90fdf5 into open-telemetry:main Sep 15, 2025
20 checks passed
@github-project-automation github-project-automation bot moved this from Next Release to Done in OTel Weaver Project Sep 15, 2025
@jsuereth jsuereth deleted the fix-864 branch January 26, 2026 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants