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

fix #28, issues with print in version 2.7 #29

Merged
merged 1 commit into from
Nov 30, 2017
Merged

Conversation

sauln
Copy link
Collaborator

@sauln sauln commented Nov 30, 2017

This implements a fix for issue #28. It seems to be caused becauseprint is a statement in 2.7 but a function in 3. Importing print function fixed the issue.

I've tested the digits example on 2.7 and 3.6 and everything seems to work fine.

@pep8speaks
Copy link

Hello @sauln! Thanks for submitting the PR.

Line 191:80: E501 line too long (80 > 79 characters)
Line 291:21: E122 continuation line missing indentation or outdented
Line 292:21: E122 continuation line missing indentation or outdented
Line 293:21: E122 continuation line missing indentation or outdented
Line 294:21: E122 continuation line missing indentation or outdented
Line 295:21: E122 continuation line missing indentation or outdented
Line 305:80: E501 line too long (80 > 79 characters)
Line 344:1: E303 too many blank lines (3)
Line 375:80: E501 line too long (80 > 79 characters)
Line 402:1: E303 too many blank lines (3)
Line 445:80: E501 line too long (80 > 79 characters)
Line 655:16: E225 missing whitespace around operator
Line 668:52: E128 continuation line under-indented for visual indent
Line 850:5: E741 ambiguous variable name 'I'
Line 868:80: E501 line too long (80 > 79 characters)
Line 1029:55: E203 whitespace before ':'
Line 1112:80: E501 line too long (80 > 79 characters)
Line 1113:9: E722 do not use bare except'
Line 1234:80: E501 line too long (80 > 79 characters)
Line 1260:80: E501 line too long (80 > 79 characters)

@lmcinnes lmcinnes merged commit 6c0c893 into lmcinnes:master Nov 30, 2017
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.

3 participants