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

Build examples last #1370

Merged
merged 1 commit into from
Dec 1, 2020
Merged

Conversation

krnowak
Copy link
Member

@krnowak krnowak commented Nov 26, 2020

Examples are "leaf" packages, that pull some other packages. So build
the other packages first to get and fix all the errors there, before
building the examples.

@krnowak
Copy link
Member Author

krnowak commented Nov 26, 2020

Not adding changelog here.

@MrAlias MrAlias added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Nov 30, 2020
Copy link
Contributor

@MrAlias MrAlias left a comment

Choose a reason for hiding this comment

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

👍

Examples are "leaf" packages, that pull some other packages. So build
the other packages first to get and fix all the errors there, before
building the examples.
@codecov
Copy link

codecov bot commented Dec 1, 2020

Codecov Report

Merging #1370 (80f955c) into master (958041d) will increase coverage by 0.0%.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1370   +/-   ##
======================================
  Coverage    77.6%   77.6%           
======================================
  Files         124     124           
  Lines        6128    6128           
======================================
+ Hits         4756    4760    +4     
+ Misses       1121    1116    -5     
- Partials      251     252    +1     
Impacted Files Coverage Δ
sdk/trace/batch_span_processor.go 78.4% <0.0%> (+3.9%) ⬆️

@MrAlias MrAlias merged commit 66db2d8 into open-telemetry:master Dec 1, 2020
@krnowak krnowak deleted the build-examples-last branch December 1, 2020 17:09
AzfaarQureshi pushed a commit to open-o11y/opentelemetry-go that referenced this pull request Dec 3, 2020
Examples are "leaf" packages, that pull some other packages. So build
the other packages first to get and fix all the errors there, before
building the examples.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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