Skip to content

Conversation

kcolton
Copy link
Contributor

@kcolton kcolton commented Feb 1, 2018

I noticed that the next page, Pipenv & Virtual Environments, re-enforces that python will be assumed to invoke the Python 3 interpreter and is Highly Preferred. The last mention on Installing Python 3 for OSX leaves python referencing the OSX System Python interpreter (2.7.10 for high Sierra).

I honestly do not know if this is the "correct" way to configure this default. It looks like brew is cautious about symlinking over the native Python 2.7.10 (High Sierra) and they might have good reason.

All of my projects are Python 3 now, so i'm ready to "make it facebook PATH official with Python 3" - as I think the documentation makes abundantly clear is a goal to to strive for. 😂

Regardless of how it's done - or if it's a bad idea and just saying sorry your SOL but use python3. This would at least remove a the discontinuity between pages and maybe help some people not go through with the native interpreter, the worst of all options on OSX.

I noticed that the very next page (which is no longer OSX specific) makes the assumption that `python` will invoke the Python 3 interpreter, while the last mention of versions on this page leaves `python` referencing the OSX System Python interpreter.

I honestly do not know if this is the "correct" way to configure this default. It looks like brew is cautious about symlinking over the native Python 2.7.10 (High Sierra) and they might have good reason. All of my projects are Python 3 now, so i'm ready to "make it ~facebook~ PATH official with Python 3"

Regardless of how it's done - or if it's a bad idea - it's worth mentioning IMO because there is a discontinuity of where this page leaves `python` and where the next page assumes `python` to be.
@kennethreitz
Copy link
Contributor

this will no longer be neccessary in about a month

@kcolton
Copy link
Contributor Author

kcolton commented Feb 4, 2018

@kennethreitz Even better! Update to brew defaults coming?

@kennethreitz
Copy link
Contributor

correct! March 1st I think :)

@kennethreitz kennethreitz merged commit a904091 into realpython:master Feb 10, 2018
@kennethreitz
Copy link
Contributor

might as well make this change in the meantime :)

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