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

Binding options #155

Merged
merged 1 commit into from
Mar 1, 2021
Merged

Conversation

atrovato
Copy link

Same as #121

@atrovato atrovato mentioned this pull request Dec 22, 2020
@atrovato atrovato force-pushed the bitcloud-pr121_binding-options branch from ee79378 to 5a99808 Compare December 22, 2020 07:47
Copy link

@rzr rzr left a 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

@atrovato atrovato mentioned this pull request Feb 6, 2021
@atrovato atrovato force-pushed the bitcloud-pr121_binding-options branch from 5a99808 to d88b9b4 Compare February 14, 2021 17:34
@atrovato atrovato marked this pull request as draft February 14, 2021 17:42
@atrovato atrovato force-pushed the bitcloud-pr121_binding-options branch 3 times, most recently from 1a79ec2 to 7b26d99 Compare February 14, 2021 17:53
@atrovato atrovato mentioned this pull request Feb 14, 2021
@atrovato atrovato force-pushed the bitcloud-pr121_binding-options branch 5 times, most recently from e8cf1f6 to 4953229 Compare February 14, 2021 19:06
@atrovato atrovato marked this pull request as ready for review February 14, 2021 20:52
@atrovato atrovato requested a review from rzr February 15, 2021 05:46
Copy link

@rzr rzr left a 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

@rzr
Copy link

rzr commented Feb 16, 2021

please rebase

@atrovato atrovato force-pushed the bitcloud-pr121_binding-options branch from 4953229 to fd0cfdb Compare February 16, 2021 20:04
@atrovato
Copy link
Author

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');
Copy link

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

Copy link

@rzr rzr left a 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 ?

@atrovato
Copy link
Author

Do you mean #182 ?

@atrovato atrovato force-pushed the bitcloud-pr121_binding-options branch from fd0cfdb to 311044b Compare February 23, 2021 19:05
@atrovato atrovato force-pushed the bitcloud-pr121_binding-options branch from 311044b to 2ce4a16 Compare February 23, 2021 20:50
Copy link

@rzr rzr left a 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 ?

@rzr rzr merged commit 743b975 into abandonware:master Mar 1, 2021
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.

2 participants