Skip to content

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
@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