-
Notifications
You must be signed in to change notification settings - Fork 160
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
Binding options #155
Binding options #155
Conversation
ee79378
to
5a99808
Compare
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.
code seems ok , but this change should be split in several PR, please start to submit linting changes
5a99808
to
d88b9b4
Compare
1a79ec2
to
7b26d99
Compare
e8cf1f6
to
4953229
Compare
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.
double check welcome from other @abandonware/reviewers
please rebase |
4953229
to
fd0cfdb
Compare
Done. |
README.md
Outdated
If you are using multiple HCI devices in one setup you can run two instances of noble with different binding configurations by initializing them seperatly in code: | ||
|
||
``` | ||
const HCIBindings = require('noble/lib/hci-socket/bindings'); |
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.
shouldnt it align to require('@abandonware/noble'); ?
or otherwise please add:
npm install bleno@npm:@abandonware/bleno
and remote @abanonware prefix elsewhere
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.
may lint changes be isolated in separate pr ?
Do you mean #182 ? |
fd0cfdb
to
311044b
Compare
311044b
to
2ce4a16
Compare
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.
LGTM what about other @abandonware/reviewers ?
Same as #121