Skip to content
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.

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

Fix makefile ci target and coverage test packages #1634

Merged
merged 1 commit into from
Mar 3, 2021

Conversation

MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Mar 3, 2021

No description provided.

@MrAlias MrAlias added bug Something isn't working Skip Changelog PRs that do not require a CHANGELOG.md entry labels Mar 3, 2021
@codecov
Copy link

codecov bot commented Mar 3, 2021

Codecov Report

Merging #1634 (9086427) into main (841d2a5) will increase coverage by 0.0%.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #1634   +/-   ##
=====================================
  Coverage   77.6%   77.7%           
=====================================
  Files        128     128           
  Lines       6639    6637    -2     
=====================================
  Hits        5157    5157           
+ Misses      1235    1233    -2     
  Partials     247     247           
Impacted Files Coverage Δ
exporters/trace/jaeger/env.go 100.0% <0.0%> (ø)
exporters/otlp/otlpgrpc/connection.go 88.7% <0.0%> (+1.8%) ⬆️

@Aneurysm9 Aneurysm9 merged commit 712c3dc into open-telemetry:main Mar 3, 2021
@MrAlias MrAlias deleted the makefile-fixes branch March 3, 2021 18:29
ldelossa pushed a commit to ldelossa/opentelemetry-go that referenced this pull request Mar 5, 2021
This was referenced Mar 7, 2021
This was referenced Mar 11, 2021
@pellared pellared added this to the untracked milestone Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants