Use NODE_BINARY as per https://github.com/getsentry/sentry-react-native/pull/2805 #379
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
…
Checklist
Why
When running a build from XCode, if the node path is non standard (for instance set with a
nix develop
environment) the expected path should be set in.xcode.env
or.xcode.env.local
and scripts should use$NODE_BINARY
instead ofnode
.@sentry/react-native
already implemented this here getsentry/sentry-react-native#2805 so this expo wrapper should also use$NODE_BINARY
.How
Replace the 2 instances of
node
by$NODE_BINARY
.Test Plan
Tested locally.