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.

Sign up for GitHub

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

Jump to bottom

OpenzwaveAdapter::init now returns an Arc #524

Merged
merged 1 commit into from
Jul 12, 2016
Merged

OpenzwaveAdapter::init now returns an Arc #524

merged 1 commit into from
Jul 12, 2016

Conversation

julienw
Copy link
Contributor

@julienw julienw commented Jun 9, 2016

No description provided.

@julienw
Copy link
Contributor Author

julienw commented Jun 9, 2016

Needed for fxbox/openzwave-stateful-rust#20

@julienw julienw mentioned this pull request Jun 9, 2016
Merged
@julienw julienw closed this Jun 20, 2016
@julienw julienw reopened this Jun 20, 2016
@julienw julienw force-pushed the openzwave-init-returns-arc branch from e2eb39a to af5982c Compare June 20, 2016 13:00
@coveralls
Copy link

coveralls commented Jun 20, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 32.51% when pulling af5982c on julienw:openzwave-init-returns-arc into 7958cc3 on fxbox:master.

@julienw
Copy link
Contributor Author

julienw commented Jun 24, 2016

I filed a bug at rsnotify: notify-rs/notify#82

@julienw julienw force-pushed the openzwave-init-returns-arc branch 2 times, most recently from 6b3b498 to 2bf3cc4 Compare June 28, 2016 13:58
@coveralls
Copy link

coveralls commented Jun 28, 2016 •
edited
Loading

Coverage Status

Coverage increased (+0.02%) to 32.479% when pulling 2bf3cc4 on julienw:openzwave-init-returns-arc into ab1dc85 on fxbox:master.

@julienw julienw force-pushed the openzwave-init-returns-arc branch 4 times, most recently from 0582de9 to d7907e3 Compare July 7, 2016 12:19
@julienw julienw force-pushed the openzwave-init-returns-arc branch from d7907e3 to 2a46393 Compare July 7, 2016 12:58
@coveralls
Copy link

Coverage Status

Coverage remained the same at 32.459% when pulling 2a46393 on julienw:openzwave-init-returns-arc into a950cda on fxbox:master.

@julienw
Copy link
Contributor Author

julienw commented Jul 7, 2016

r? @JohanLorenzo @fabricedesre

likely my final patch in foxbox :D

@JohanLorenzo
Copy link
Contributor

LGTM!

@fabricedesre
Copy link
Collaborator

r=me

@julienw julienw merged commit a2b0d90 into fxbox:master Jul 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@julienw @coveralls @JohanLorenzo @fabricedesre