Skip to content

Merge observability report sender for all signals, remove duplicate code #12165

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.

Sign up for GitHub

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

Merged
merged 1 commit into from
Jan 23, 2025

Merge observability report sender for all signals, remove duplicate code

92defee
Select commit
Loading
Failed to load commit list.
Merged

Merge observability report sender for all signals, remove duplicate code #12165

Merge observability report sender for all signals, remove duplicate code
92defee
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Jan 23, 2025 in 0s

92.77% of diff hit (target 95.00%)

View this Pull Request on Codecov

92.77% of diff hit (target 95.00%)

Annotations

Check warning on line 59 in exporter/exporterhelper/internal/obs_report.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

exporter/exporterhelper/internal/obs_report.go#L58-L59 

Added lines #L58 - L59 were not covered by tests

Check warning on line 138 in exporter/exporterhelper/internal/obs_report.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

exporter/exporterhelper/internal/obs_report.go#L137-L138 

Added lines #L137 - L138 were not covered by tests
View more details on Codecov