Skip to content
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 new autoOpen option, defaults to true #178

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

Conversation

sandeepmistry
Copy link
Contributor

Inspired by node-serialport's open options, this change allows you to delay opening the HID device until a new open method is called.

If the proposal looks good, I can update the read me accordingly.

@todbot
Copy link
Contributor

todbot commented Jan 24, 2017

Looks great to me. I'd love to see examples of how this could be useful. Thanks!

@sandeepmistry
Copy link
Contributor Author

For sure, stay tuned!

My main goal is to shim the Chrome HID API's so I can use node-hid API's and dependent modules in both Node.js and a Chrome app. I'm going to experiment with adding optional callback arguments to various methods next.

Some examples of where something similar has been done before are:

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.

None yet

2 participants