We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
splice()
argv
I'm not sure if possibly the contents of argv changed since this sample was published, but the current impl produces the wrong outcome.
When testing, I observed that argv contains:
["/usr/local/bin/node","/home/pptruser/screenshot.js","screenshot.js","https://example.com","https://cloud.google.com"]
This line which uses splice(2), should probably be splice(3)
splice(2)
splice(3)
jobs-demos/screenshot/screenshot.js
Line 97 in bae6dc8
The text was updated successfully, but these errors were encountered:
Or switch to node.js 20 and use parseArgs, the new built-in CLI flag parser
parseArgs
Sorry, something went wrong.
No branches or pull requests
I'm not sure if possibly the contents of
argv
changed since this sample was published, but the current impl produces the wrong outcome.When testing, I observed that argv contains:
["/usr/local/bin/node","/home/pptruser/screenshot.js","screenshot.js","https://example.com","https://cloud.google.com"]
This line which uses
splice(2)
, should probably besplice(3)
jobs-demos/screenshot/screenshot.js
Line 97 in bae6dc8
The text was updated successfully, but these errors were encountered: