Skip to content

Conversation

dims
Copy link
Collaborator

@dims dims commented Jan 12, 2017

Add a travis job for docs. This will make sure we can catch if a PR is likely to break docs

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jan 12, 2017
@mbohlool mbohlool self-assigned this Jan 12, 2017
@codecov-io
Copy link

codecov-io commented Jan 12, 2017

Current coverage is 94.52% (diff: 100%)

Merging #96 into master will not change coverage

@@             master        #96   diff @@
==========================================
  Files             9          9          
  Lines           658        658          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits            622        622          
  Misses           36         36          
  Partials          0          0          

Powered by Codecov. Last update 37d20f2...8d4cd14

.travis.yml Outdated
- TOXENV=py27,codecov
- TOXENV=py27-functional,codecov
- TOXENV=py35-functional,codecov
- TOXENV=docs,codecov
Copy link
Contributor

Choose a reason for hiding this comment

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

do we need codecov for all of these? I think it will fail for doc or functional tests. Also we have a coverage env, maybe that is enough?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yeah, i'll fix it in another PR @mbohlool

@mbohlool
Copy link
Contributor

you have a conflict. I guess now that you are fixing the conflict you can remove all unnecessary codecov commands. Up to you though. this look good to me after fixing the conflict.

@mbohlool mbohlool added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 13, 2017
@dims dims force-pushed the enforce-doc-builds branch from a2eb6c3 to 8d4cd14 Compare January 13, 2017 10:57
@dims
Copy link
Collaborator Author

dims commented Jan 13, 2017

Ack @mbohlool fixing it while i fix the conflicts in this PR

@dims
Copy link
Collaborator Author

dims commented Jan 13, 2017

@mbohlool - rebased. Let's get this in please.

@mbohlool
Copy link
Contributor

Great. Thanks.

@mbohlool mbohlool merged commit c8b0968 into kubernetes-client:master Jan 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants