-
Notifications
You must be signed in to change notification settings - Fork 68
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
Add setting to allow user to change timeout. #900
Conversation
package.json
Outdated
@@ -765,6 +765,11 @@ | |||
"description": "Command to execute or full path to folder and executable to start the Mongo shell, needed by some Mongo scrapbook commands. If empty, will search in the system path for 'mongo'.", | |||
"default": null | |||
}, | |||
"mongo.shell.timeoutInMilliseconds": { | |||
"type": "number", | |||
"description": "The duration allowed (in milliseconds) for the mongo shell to execute a command. Default value is 5 seconds", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mongo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like "inMilliseconds" for variable names, but not so sure users will like it for options. I'd go with simple mongo.shell.timeout
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mongo in 'Mongo shell'?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you're going to get rid of "inMilliseconds" I would make the timeout based on seconds. I feel like that's much more common for these types of settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixes #809