Skip to content

Comments

Make compatible with pnpm#14

Open
AJamesPhillips wants to merge 2 commits intomtiller:masterfrom
AJamesPhillips:pnpm_compatible
Open

Make compatible with pnpm#14
AJamesPhillips wants to merge 2 commits intomtiller:masterfrom
AJamesPhillips:pnpm_compatible

Conversation

@AJamesPhillips
Copy link

When using the pnpm package manager and running this launch.json config it was failing with:

ts-jest-sample %  cd /Users/my_user/ts-jest-sample ; /usr/bin/env 'NOD
E_OPTIONS= --require "/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/ms-vscode.js-debug/src/bootloader.
js"  --inspect-publish-uid=http' 'VSCODE_INSPECTOR_OPTIONS=:::{"inspectorIpc":"/var/folders/v6/j9r9l99d7nv36b1jppx8j1mh0000gn/T
/node-cdp.75090-b7cacc76-7.sock","deferredMode":false,"waitForDebugger":"","execPath":"/Users/my_user/.nvm/versions/node/v20.12.2/b
in/node","onlyEntrypoint":false,"autoAttachMode":"always","fileCallback":"/var/folders/v6/j9r9l99d7nv36b1jppx8j1mh0000gn/T/node
-debug-callback-b52a049f5ebf242c"}' /Users/my_user/.nvm/versions/node/v20.12.2/bin/node /Users/my_user/ts-jest-sampl
e/node_modules/.bin/jest --runInBand 
Debugger attached.
Waiting for the debugger to disconnect...
/Users/my_user/ts-jest-sample/node_modules/.bin/jest:2
basedir=$(dirname "$(echo "$0" | sed -e 's,\\,/,g')")
          ^^^^^^^

SyntaxError: missing ) after argument list
    at internalCompileFunction (node:internal/vm:128:18)
    at wrapSafe (node:internal/modules/cjs/loader:1280:20)
    at Module._compile (node:internal/modules/cjs/loader:1332:27)
    at Module._extensions..js (node:internal/modules/cjs/loader:1427:10)
    at Module.load (node:internal/modules/cjs/loader:1206:32)
    at Module._load (node:internal/modules/cjs/loader:1022:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:135:12)
    at node:internal/main/run_main_module:28:49

Node.js v20.12.2

When I changed the launch.json to that in this PR (and upgraded the dependencies in #13) then this error went away.

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.

1 participant