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

Remove deploy docs step from Travis #8

Merged
merged 1 commit into from
Apr 8, 2018

Conversation

isaachier
Copy link
Contributor

No description provided.

@codecov-io
Copy link

codecov-io commented Apr 5, 2018

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master     #8   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           2      2           
  Lines         123    123           
=====================================
  Hits          123    123

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 d2de7cd...a6be514. Read the comment docs.

@isaachier isaachier requested a review from yurishkuro April 5, 2018 13:53
@isaachier
Copy link
Contributor Author

isaachier commented Apr 5, 2018

I'm gonna try deploy keys before merging, but this is an alternative to remove the step altogether.

@isaachier
Copy link
Contributor Author

I'm just going ahead with this and leaving #7 open until a someone finds a solution.

@isaachier isaachier merged commit 9895329 into opentracing:master Apr 8, 2018
@isaachier isaachier deleted the travis-remove-deploy-fix branch April 8, 2018 17:10
@ror6ax
Copy link

ror6ax commented Apr 8, 2018

@isaachier - can you please detail what you want done here in #7 ? I can have a look into this, been using travis a lot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants