-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
Make pytest run the doctests in doc rst files #9445
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
Comments
Setup.cfg does specify --doctest-modules, and test_script.sh appears to
install it... Not sure what's going on here.
…On 25 Jul 2017 1:57 am, "Andreas Mueller" ***@***.***> wrote:
It looks like the pytest run on travis doesn't run the doctests:
https://travis-ci.org/scikit-learn/scikit-learn/builds/
256859150?utm_source=github_status&utm_medium=notification
We should fix that.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#9445>, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/AAEz61TcXqZoqAsMErdcSKvHf56PNs1fks5sRL7sgaJpZM4OhY3k>
.
|
This was noted in one of the pytest issue/PR and it something that is on my radar once we released 0.19 and move fully to pytest. |
e.g. in #8246 (comment). |
I think this issue is about the doctests in the doc .rst not the doctests in the python modules. |
@lesteve good radar :) |
I'm not sure I understand your comment. It seems to work fine for me locally, using |
Interestingly it works if I run it in the modules subfolder, but running it in the |
Interestingly
results in multiple ImportMismatchErrors for me locally... |
Off the top of my head one thing I remember last time I looked is that there is no straight equivalent of nosetests I did not look at it in more details since. |
you have to call |
It looks like the pytest run on travis doesn't run the doctests:
https://travis-ci.org/scikit-learn/scikit-learn/builds/256859150?utm_source=github_status&utm_medium=notification
We should fix that.
The text was updated successfully, but these errors were encountered: