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

chore: onboard onto Python 3.12-dev and Node 18 #201

Merged
merged 16 commits into from
Aug 26, 2023

Conversation

rzhao271
Copy link
Contributor

@rzhao271 rzhao271 commented Jul 29, 2023

Combines #196 and nodejs/node-gyp#2888

@rzhao271 rzhao271 changed the title Debug node-gyp tests run with Python 3.12-dev and Node 18 Get node-gyp to run with Python 3.12-dev and Node 18 Jul 29, 2023
@rzhao271
Copy link
Contributor Author

I notice the (windows-latest, 3.8) integration tests contain failed tests, but the error message says gyp verb find Python - version is "3.11.4". @cclauss Is node-gyp not finding Python 3.8 properly?

Other than that, CI looks a lot more green than before.

@cclauss
Copy link
Contributor

cclauss commented Jul 30, 2023

If you have a later version of Python installed then node-gyp may try to use it.
If py3.11 is disabled before the test runs, it may find 3.8 and pass.

@rzhao271
Copy link
Contributor Author

rzhao271 commented Aug 26, 2023

@cclauss PR finally ready to merge.

Reminder: please squash merge the PR.

Afterwards, can #197 be updated and merged to unblock node-gyp? I'm thinking the other PRs such as lint PRs can be merged later.

@cclauss cclauss merged commit 874233e into nodejs:main Aug 26, 2023
20 checks passed
@rzhao271 rzhao271 deleted the rzhao271/bump-12 branch August 26, 2023 00:09
@rzhao271 rzhao271 changed the title Get node-gyp to run with Python 3.12-dev and Node 18 chore: onboard onto Python 3.12-dev and Node 18 Aug 27, 2023
cclauss added a commit to nodejs/node-gyp that referenced this pull request Sep 14, 2023
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