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

check ch02 on latest softw. #103

Merged
merged 13 commits into from
Dec 5, 2019
Merged

check ch02 on latest softw. #103

merged 13 commits into from
Dec 5, 2019

Conversation

rasbt
Copy link
Owner

@rasbt rasbt commented Dec 5, 2019

Last check before finalizing the PDFs.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

You'll be able to see Jupyter notebook diff and discuss changes. Powered by ReviewNB.

Verified

This commit was signed with the committer’s verified signature. The key has expired.
re-taro Rintaro Itokawa
@pep8speaks
Copy link

pep8speaks commented Dec 5, 2019

Hello @rasbt! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 38:1: E303 too many blank lines (5)
Line 38:80: E501 line too long (82 > 79 characters)
Line 38:83: W291 trailing whitespace
Line 39:2: W291 trailing whitespace
Line 40:45: W291 trailing whitespace
Line 41:2: W291 trailing whitespace
Line 42:5: W291 trailing whitespace
Line 43:2: W291 trailing whitespace
Line 44:28: W291 trailing whitespace
Line 45:2: W291 trailing whitespace
Line 638:1: E265 block comment should start with '# '

Line 323:1: E303 too many blank lines (3)
Line 325:80: E501 line too long (375 > 79 characters)

Line 174:1: E303 too many blank lines (9)
Line 343:28: W291 trailing whitespace

Line 157:52: W291 trailing whitespace

Comment last updated at 2019-12-05 07:25:40 UTC

@rasbt rasbt merged commit 83420ff into master Dec 5, 2019
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.

None yet

2 participants