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

[Builder]: Remove test that depends on the contrib #4787

Merged
merged 1 commit into from
Feb 26, 2022

Conversation

bogdandrutu
Copy link
Member

@bogdandrutu bogdandrutu commented Feb 1, 2022

Remove unnecessary test for replace since the cmd/otelcorecol tests the latest binary with replace statements already.

Signed-off-by: Bogdan Drutu [email protected]

@bogdandrutu bogdandrutu requested review from a team and dmitryax February 1, 2022 02:29
@codecov
Copy link

codecov bot commented Feb 1, 2022

Codecov Report

Merging #4787 (1f4a4dd) into main (d21d20b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4787   +/-   ##
=======================================
  Coverage   90.82%   90.82%           
=======================================
  Files         181      181           
  Lines       10632    10632           
=======================================
  Hits         9657     9657           
  Misses        759      759           
  Partials      216      216           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d21d20b...1f4a4dd. Read the comment docs.

@bogdandrutu
Copy link
Member Author

/cc @jpkrohling

@codeboten codeboten added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Feb 2, 2022
@bogdandrutu bogdandrutu changed the title Always test with latest core/model to ensure we can build at head [Builder]: Test with current head and latest release Feb 14, 2022
@bogdandrutu bogdandrutu force-pushed the test2 branch 2 times, most recently from d1a36f8 to c8072be Compare February 14, 2022 17:50
@bogdandrutu bogdandrutu force-pushed the test2 branch 2 times, most recently from 0b19d20 to 1fcef17 Compare February 19, 2022 18:17
The latest is tested by the cmd/otelcorecol which uses the builder to generate the files.

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu bogdandrutu changed the title [Builder]: Test with current head and latest release [Builder]: Remove test that depends on the contrib Feb 25, 2022
@bogdandrutu bogdandrutu merged commit 4998703 into open-telemetry:main Feb 26, 2022
@bogdandrutu bogdandrutu deleted the test2 branch February 26, 2022 15:34
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.

3 participants