Skip to content

Several modifications#367

Merged
sigmavirus24 merged 6 commits into
realpython:masterfrom
cli248:update
Feb 12, 2014
Merged

Several modifications#367
sigmavirus24 merged 6 commits into
realpython:masterfrom
cli248:update

Conversation

@cli248
Copy link
Copy Markdown
Contributor

@cli248 cli248 commented Feb 12, 2014

See details on the changed files.

Comment thread docs/intro/learning.rst
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please collapse remove this and make This book at the beginning of the preceding paragraph the link.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @sigmavirus24, it seems the other suggested books, i.e., Python Koans, A Byte of Python, Expert Python, they don't have the link at the beginning of the preceding paragraph. That is why I put the Pro Python link at the line 107, which is consistent with other books.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, good call. I had forgotten about that.

@sigmavirus24
Copy link
Copy Markdown
Contributor

In the future, I would appreciate more PRs with a narrower focus personally. That aside, thanks for this! 🍰

sigmavirus24 added a commit that referenced this pull request Feb 12, 2014
@sigmavirus24 sigmavirus24 merged commit 09de7cb into realpython:master Feb 12, 2014
@cli248 cli248 deleted the update branch February 12, 2014 22:26
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