Skip to content

Commit

Permalink
Node 11 is fixed, so let’s remove this.
Browse files Browse the repository at this point in the history
When a script is invoked by yarn, it provides the appropriate -r via NODE_OPTIONS. This makes all node scripts invoke via yarn automatically pnp aware. 

Unfortunately NODE_OPTIONS was partially broken in node 11, preventing this from working.
Luckily node 11.12 fixed this issue.

Relevant issue: nodejs/node#26521
  • Loading branch information
stefanpenner committed Mar 23, 2019
1 parent 4eb8bad commit c557474
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/index-test.ts
Expand Up @@ -80,7 +80,7 @@ describe('resolve-package-path', function() {
app.pkg.private = true;
app.pkg.name
app.pkg.scripts = {
test: "node -r ./.pnp.js ./test.js"
test: "node ./test.js"
};
app.pkg.installConfig = {
pnp: true
Expand Down

0 comments on commit c557474

Please sign in to comment.