-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Support different node flags for different versions of node. #18070
Conversation
912be29
to
1ab17d7
Compare
ed1e41f
to
f7397fc
Compare
9de4d22
to
20d8535
Compare
5147440
to
73c4535
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
.circleci/config.yml
Outdated
@@ -6,7 +6,10 @@ orbs: | |||
executors: | |||
linux-node: | |||
docker: | |||
- image: circleci/node:stretch | |||
- image: cimg/node:19.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need both the docker image for node 19 and also to download it manually below?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is actually a separate change, I think i'll just revert it. we don't run these new tests in the linux-node
image.
73c4535
to
9feca29
Compare
This is mostly about what flags to pass to node during testing, but also applies to `CMAKE_CROSSCOMPILING_EMULATOR` and to the #! used in autoconf tests.
9feca29
to
cf50cfa
Compare
This is mostly about what flags to pass to node during testing, but also applies to
CMAKE_CROSSCOMPILING_EMULATOR
and to the #! used in autoconf tests.