Skip to content

PIL Link Fix #570

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

Merged
merged 1 commit into from
Oct 2, 2015
Merged

PIL Link Fix #570

merged 1 commit into from
Oct 2, 2015

Conversation

varunagrawal
Copy link
Contributor

  1. Added Dive Into Python for those looking to learn Python 2 first.
  2. Fix for Broken PIL installation link #569

@sigmavirus24
Copy link
Contributor

Pull requests should be single purposed. Please only fix the PIL link for this. That is desirable.

If my memory serves me well, Dive Into Python covers Python 2.4 (or maybe 2.5) which is very out of date. I don't think we should be suggesting it at all in the Guide for that reason.

@varunagrawal varunagrawal changed the title Additions and Fixes PIL Link Fix Oct 2, 2015
@varunagrawal
Copy link
Contributor Author

Updated to only a single commit.
Also, I just went through the book, and Mr. Pilgrim recommends installing the latest version Python everywhere. There doesn't seem to be a dependency on a particular version of Python 2.x.

@sigmavirus24
Copy link
Contributor

Even so, it's not appropriate to jam everything into one pull request.

sigmavirus24 added a commit that referenced this pull request Oct 2, 2015
@sigmavirus24 sigmavirus24 merged commit 06fcb71 into realpython:master Oct 2, 2015
@varunagrawal
Copy link
Contributor Author

Yes I agree, I am sorry about that. Still not completely comfortable with how Pull Requests work. :(

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