Skip to content

RF: remove doctest string markup #369

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

Merged
merged 1 commit into from
Nov 6, 2015

Conversation

matthew-brett
Copy link
Member

The doctest string markup was for allowing doctests running on Python 3
and Python < 2.6. We haven't used it for a few years. This commit
removes that code in the hope that no-one else is using it.

The doctest string markup was for allowing doctests running on Python 3
and Python < 2.6.   We haven't used it for a few years.  This commit
removes that code in the hope that no-one else is using it.
@matthew-brett
Copy link
Member Author

A google and OpenHub search for uses of the nisext package and py3builder doesn't come up with anything outside our little world, so I think this is safe to merge.

@effigies
Copy link
Member

No objections.

@matthew-brett
Copy link
Member Author

Last call for comments?

matthew-brett added a commit that referenced this pull request Nov 6, 2015
MRG: remove doctest string markup

The doctest string markup was for allowing doctests running on Python 3
and Python < 2.6. We haven't used it for a few years. This commit
removes that code in the hope that no-one else is using it.
@matthew-brett matthew-brett merged commit ca19fa5 into nipy:master Nov 6, 2015
@effigies effigies mentioned this pull request Nov 6, 2015
grlee77 pushed a commit to grlee77/nibabel that referenced this pull request Mar 15, 2016
MRG: remove doctest string markup

The doctest string markup was for allowing doctests running on Python 3
and Python < 2.6. We haven't used it for a few years. This commit
removes that code in the hope that no-one else is using it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants