Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
nose
was still expected byconda build
to run the test suite, which couldn't work at all since it was replaced recently by pytest.Part of this PR I've also decided to move the last doctest (param.Time) to a unit test. Why? Because conda build failed by simply replacing
nosetests tests
bypytest tests
. It executes this from a folder that only includes the subfoldertests
, and the command that was set to tell pytest to look for doctest to execute required the subfolders param and numberben to be also there (pytest tests param numbergen --doc-modules).There may have been a way to configure pytest to cover this case but I thought it was just simpler to remove that last doctest. This makes even more sense now since param has some nice docs :)
Note that I've not removed the docstest code from the docstring, since it is still useful docs in itself. It's just no longer executed by pytest.