Skip to content

docs: add info about --auto-open flag w/ --inspect #7992

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

Closed
wants to merge 1 commit into from

Conversation

kentcdodds
Copy link

Checklist
  • make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

This feature doesn't actually work (AFAIK). This is mostly just my way of suggesting that we implement a CLI flag for it. I'd be happy to help with a little direction (first time contributor here).

This feature doesn't actually work (AFAIK). This is mostly just my way of suggesting that we implement a CLI flag for it. I'd be happy to help with a little direction (first time contributor here).
@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. debugger labels Aug 6, 2016
@princejwesley
Copy link
Contributor

princejwesley commented Aug 6, 2016

@kentcdodds --auto-open ?

node 🙈  git:(master 103) ./node --version
v7.0.0-pre
node 🙈  git:(master 103) ./node --debug-brk --inspect --auto-open ~/Desktop/test.js
node: bad option: --auto-open

~ 🙈  node --version
v6.3.1
~ 🙈  node --debug-brk --inspect --auto-open ~/Desktop/test.js
node: bad option: --auto-open

Edit: Sorry, I just read the commit.

@mscdex mscdex added inspector Issues and PRs related to the V8 inspector protocol feature request Issues that request new features to be added to Node.js. and removed debugger doc Issues and PRs related to the documentations. labels Aug 6, 2016
@rvagg rvagg force-pushed the master branch 2 times, most recently from c133999 to 83c7a88 Compare October 18, 2016 17:02
@jasnell
Copy link
Member

jasnell commented Mar 1, 2017

Is this still needed?

@jasnell jasnell added the stalled Issues and PRs that are stalled. label Mar 1, 2017
@kentcdodds
Copy link
Author

I'd still love this flag/functionality to exist (if it's not already supported)

@fhinkel
Copy link
Member

fhinkel commented Mar 26, 2017

@kentcdodds This would be a great feature! (I'm using Node.js V8 inspector, but it's still one extra click.)

As a PR, I think this is not getting anywhere. Can you open an issue instead? We'll attach the feature request label and see if anybody has a solution.

Thanks!

@fhinkel fhinkel closed this Mar 26, 2017
@kentcdodds kentcdodds deleted the pr/auto-open branch March 27, 2017 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Issues that request new features to be added to Node.js. inspector Issues and PRs related to the V8 inspector protocol stalled Issues and PRs that are stalled.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants