Skip to content

build: recommend Python 3.8 to build on Windows - #34182

Closed
targos wants to merge 2 commits into
nodejs:masterfrom
targos:python-38-win
Closed

build: recommend Python 3.8 to build on Windows#34182
targos wants to merge 2 commits into
nodejs:masterfrom
targos:python-38-win

Conversation

@targos

Copy link
Copy Markdown
Member

@nodejs/python @nodejs/platform-windows

@nodejs-github-botnodejs-github-bot added build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project. labels Jul 3, 2020
Comment thread.github/workflows/build-tarball.yml Outdated

@cclausscclauss left a comment

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.

Excellent!!

Comment threadBUILDING.md
##### Option 1: Manual install

* [Python 2.7](https://www.python.org/downloads/)
* [Python 3.8](https://www.python.org/downloads/)

@cclausscclaussJul 3, 2020

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.

Why not https://www.microsoft.com/en-us/p/python-38/9mssztt1n39l ? It will prompt users as updates become available.

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

I don't know if it's available on Windows 8

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think no. The System Requirements on MS store said only for Windows10

Co-authored-by: Christian Clauss <cclauss@me.com>
targos added a commit that referenced this pull request Jul 13, 2020
PR-URL: #34182
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
@targos

Copy link
Copy Markdown
MemberAuthor

Landed in 913d36d

@targostargos closed this Jul 13, 2020
@targos
targos deleted the python-38-win branch July 13, 2020 09:52
MylesBorins pushed a commit that referenced this pull request Jul 14, 2020
PR-URL: #34182
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Jul 14, 2020
MylesBorins pushed a commit that referenced this pull request Jul 16, 2020
PR-URL: #34182
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buildIssues and PRs related to build files or the CI.docIssues and PRs related to the documentations.metaIssues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@targos@jasnell@lpinca@tniessen@cclauss@gengjiawen@himself65@nodejs-github-bot