-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Test if docs can be built #96
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
Conversation
Current coverage is 94.52% (diff: 100%)@@ 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
|
@@ -9,6 +9,7 @@ env: | |||
- TOXENV=py27,codecov | |||
- TOXENV=py27-functional,codecov | |||
- TOXENV=py35-functional,codecov | |||
- TOXENV=docs,codecov |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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
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. |
a2eb6c3
to
8d4cd14
Compare
Ack @mbohlool fixing it while i fix the conflicts in this PR |
@mbohlool - rebased. Let's get this in please. |
Great. Thanks. |
Add a travis job for docs. This will make sure we can catch if a PR is likely to break docs