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

STYLE: PEP8 for workflows folder #896

Closed
wants to merge 11 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@kennyho-code

kennyho-code commented Feb 7, 2016

STYLE: pep8 fixes for workflow folder.

kennyho-code added some commits Feb 7, 2016

STYLE: PEP 8 workflows
PEP8 for  dipy/workflows/tests/test_docstring_parser.py
STYLE: pep8 workflows
pep 8 for dipy/workflows/tests/test_docstring_parser.py
STYLE: pep8 workflows
pep8 for dipy/workflows/tests/test_docstring_parser.py
STYLE: pep8 workflows
pep8 for dipy/workflows/tests/test_iap.py
STYLE: pep8 workflows
pep8 for dipy/workflows/tests/test_utils.py
STYLE: PEP8 workflows
pep8 for dipy/workflows/base.py
STYLE: PEP8 workflows
pep8 for dipy/workflows/docstring_parser.py
STYLE: PEP8 workflows
pep8 for dipy/workflows/utils.py
STYLE: PEP8 workflows
pep8 for dipy/__init__.py
STYLE: aligned '='
no pep8. alignment for ‘=‘ for better style in dipy/info.py
STYLE: PEP8 workflows
pep8 for dipy/pkg_info.py

@kennyho-code kennyho-code changed the title from Pep8workflows to PEP8 for workflows folder Feb 7, 2016

@kennyho-code kennyho-code changed the title from PEP8 for workflows folder to STYLE: PEP8 for workflows folder Feb 7, 2016

@arokem arokem referenced this pull request Feb 8, 2016

Closed

Pep8 #891 #898

@arokem

This comment has been minimized.

Member

arokem commented Feb 8, 2016

Nice work! Still remaining for these files:

dipy/workflows/tests/test_iap.py:33:12: E127 continuation line over-indented for visual indent
dipy/workflows/tests/test_iap.py:34:12: E127 continuation line over-indented for visual indent

You might want to use the pep8 program (https://pypi.python.org/pypi/pep8) or a pep8-enabled editor (https://atom.io/packages/linter-pep8) to check.

This was referenced Feb 12, 2016

@arokem

This comment has been minimized.

Member

arokem commented Feb 12, 2016

For #896

@arokem

This comment has been minimized.

Member

arokem commented Feb 14, 2016

@hokatvcu - did you mean to close this, or do you want us to merge this?

@kennyho-code

This comment has been minimized.

kennyho-code commented Feb 14, 2016

close it please.

@sahmed95

This comment has been minimized.

Contributor

sahmed95 commented Feb 19, 2016

#930 pep8 fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment