Skip to content
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

PEP8 #897

Merged
merged 2 commits into from
Mar 3, 2016
Merged

PEP8 #897

merged 2 commits into from
Mar 3, 2016

Conversation

omarocegueda
Copy link
Contributor

Closes issue #862. Verified with Spyder.

@arokem
Copy link
Contributor

arokem commented Feb 8, 2016

Thanks for taking a look. Spyder is apparently not aware of this one:

dipy/align/tests/test_crosscorr.py:40:5: E265 block comment should start with '# '
dipy/align/tests/test_crosscorr.py:44:5: E265 block comment should start with '# '
dipy/align/tests/test_crosscorr.py:48:5: E265 block comment should start with '# '
dipy/align/tests/test_crosscorr.py:56:5: E265 block comment should start with '# '
dipy/align/tests/test_crosscorr.py:85:5: E265 block comment should start with '# '
dipy/align/tests/test_crosscorr.py:89:5: E265 block comment should start with '# '
dipy/align/tests/test_crosscorr.py:110:5: E265 block comment should start with '# '
dipy/align/tests/test_crosscorr.py:130:5: E265 block comment should start with '# '
dipy/align/tests/test_crosscorr.py:134:5: E265 block comment should start with '# '
dipy/align/tests/test_crosscorr.py:144:5: E265 block comment should start with '# '
dipy/align/tests/test_crosscorr.py:172:5: E265 block comment should start with '# '
dipy/align/tests/test_crosscorr.py:176:5: E265 block comment should start with '# '
dipy/align/tests/test_crosscorr.py:199:5: E265 block comment should start with '# '

@arokem
Copy link
Contributor

arokem commented Feb 8, 2016

To be clear, the hash sign (#) should be followed by space before the text. So:

#Select arbitrary images' shape (same shape for both images)

Should be:

# Select arbitrary images' shape (same shape for both images)

@arokem
Copy link
Contributor

arokem commented Mar 1, 2016

Hey @omarocegueda - any chance to finish this up?

@omarocegueda
Copy link
Contributor Author

Sorry @arokem, I didn't see your comment! I just fixed the remaining issues.
You are right, Spyder didn't catch that, I also tried pep8 from here:
https://pypi.python.org/pypi/pep8
and it doesn't catch it either, what tool do you use?

@omarocegueda
Copy link
Contributor Author

I just tried flake8 (following one of your previous comments) and it correctly found the issue, thanks! =)

@arokem
Copy link
Contributor

arokem commented Mar 1, 2016

Great! 👍

@arokem
Copy link
Contributor

arokem commented Mar 1, 2016

I'm going to go ahead and merge this, even though Travis isn't done, since that last commit is just innocuous whitespace.

arokem added a commit that referenced this pull request Mar 3, 2016
@arokem arokem merged commit 76d3b5f into dipy:master Mar 3, 2016
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