ENH: adding doctest-requires-all directive by bsipocz · Pull Request #280 · scientific-python/pytest-doctestplus
The need for this came up again recently in astroquery, so finally I got down to do it.
To close #183
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I wonder if this will also be useful for #279.
Just some minor comments.
|
|
||
| Multiple requirements can be specified if separated by semicolons. | ||
|
|
||
|
|
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: P. L. Lim <2090236+pllim@users.noreply.github.com>
Comment on lines +453 to +455
| This is a narrative line, before another doctest snippet | ||
|
|
||
| >>> import foobar |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix formatting to reflect typical real use case.
| This is a narrative line, before another doctest snippet | |
| >>> import foobar | |
| This is a narrative line, before another doctest snippet:: | |
| >>> import foobar |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should not matter, all the snippets should be collected no matter if there are :: or correct number of spaces before the >>>.
I go ahead and merge this now, so the other PR doesn't have to be rebased.
bsipocz
deleted the
doctest-requires-all
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters