Skip to content
This repository has been archived by the owner on Dec 19, 2022. It is now read-only.

#234 #242 Changed --debug to --inspect in node args #246

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MicroCheapFx
Copy link

In order to fix #242 #234 , I propose this commit.
Also today, current and LTS versions of node are above 8 and need this to be fixed.

Regards

@skywinder
Copy link

👍
Please kindly check

@timkinnane
Copy link

timkinnane commented Nov 24, 2017

Doesn't work for me. Tried installing the PR as follows...

git clone kiddkai/atom-node-debugger
cd atom-node-debugger
git pull origin pull/246/head
npm install
apm link
apm install

Then restarted atom, went to package settings, set the new port 9229 (5858 was the pre node v8 default port) and my custom node path (because I use nvm, had to get that with nvm which default). Now running my package with node main.js --inspect --inspect-brk and in atom Node Debugger: Attach.

No success. Connection attempt 1 to node process on 127.0.0.1:9229 failed. Will try again in 500.
:(

Let me know if I'm doing something wrong.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Getting exception for Node --debug argument
3 participants