Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

Use Travis CI #536

Closed
wants to merge 3 commits into from
Closed

Conversation

mayurkale22
Copy link
Member

@mayurkale22 mayurkale22 commented May 17, 2019

Fixes #507

@codecov-io
Copy link

codecov-io commented May 17, 2019

Codecov Report

Merging #536 into master will decrease coverage by 0.19%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #536     +/-   ##
=========================================
- Coverage   95.11%   94.92%   -0.2%     
=========================================
  Files         147      303    +156     
  Lines        9715    18544   +8829     
  Branches      527     1085    +558     
=========================================
+ Hits         9240    17602   +8362     
- Misses        475      942    +467
Impacted Files Coverage Δ
src/common/noop-logger.ts 16.66% <0%> (-3.34%) ⬇️
src/zpages-frontend/latency-bucket-boundaries.ts 68.65% <0%> (-2.99%) ⬇️
src/tags/propagation/binary-serializer.ts 95.23% <0%> (-1.13%) ⬇️
src/http-stats.ts 100% <0%> (ø) ⬆️
test/test-tag-map.ts 100% <0%> (ø) ⬆️
src/binary-format.ts 100% <0%> (ø) ⬆️
test/test-no-record-span.ts 100% <0%> (ø) ⬆️
test/test-metric-producer-manager.ts 100% <0%> (ø) ⬆️
test/test-exporter-buffer.ts 100% <0%> (ø) ⬆️
test/test-utils.ts 100% <0%> (ø) ⬆️
... and 182 more

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 a7af26c...fbe38ca. Read the comment docs.

@mayurkale22 mayurkale22 marked this pull request as ready for review May 17, 2019 21:55
Copy link
Contributor

@draffensperger draffensperger left a comment

Choose a reason for hiding this comment

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

I'm curious, what's the motivation for using Travis when we already have CircleCI for this repo?

@mayurkale22
Copy link
Member Author

I'm curious, what's the motivation for using Travis when we already have CircleCI for this repo?

OpenTelemetry is also going to use TravisCI, to make things consistent I thought to use TravisCI here. Sounds good?

@draffensperger
Copy link
Contributor

Makes sense, thanks for explaining!

@mayurkale22
Copy link
Member Author

Makes sense, thanks for explaining!

I will get rid of CircleCI, TravisCI looks stable.

@aabmass
Copy link
Member

aabmass commented Feb 15, 2021

Closing since Travis CI is not recommended for Google projects anymore. Also, OTel is using Github Actions most places now

@aabmass aabmass closed this Feb 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate to TravisCI
6 participants