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

generic-sensors-adapter: Update to v0.0.10 #650

Merged

Conversation

rzr
Copy link
Contributor

@rzr rzr commented Jan 15, 2020

Bug: rzr/generic-sensors-webthings#11
Relate-to: #54
Change-Id: I576648a50ba66abe91688359abdfa281a0ae63fa
Signed-off-by: Philippe Coval rzr@users.sf.net

Forwarded: WebThingsIO#650
Bug: rzr/generic-sensors-webthings#11
Relate-to: WebThingsIO#54
Change-Id: I576648a50ba66abe91688359abdfa281a0ae63fa
Signed-off-by: Philippe Coval <rzr@users.sf.net>
@rzr rzr force-pushed the sandbox/rzr/review/master branch from 9cea299 to 8ec8ec6 Compare January 15, 2020 17:00
@mrstegeman
Copy link
Contributor

Thanks for taking care of this!

Would you have a problem with me pulling this into the addon-builder, so that we can get builds for Node 10 and 12 as well? This is one of 4 remaining add-ons that will only work with Node 8.

@rzr
Copy link
Contributor Author

rzr commented Jan 15, 2020

I can do that or if you want to, go ahead,
as you know I used my own builder script (and it's really slow)

node 10 should work using the updated @abandonware i2c module
I didnt look at 12 yet

Anyway I will probably a new version before next release

@mrstegeman
Copy link
Contributor

Ok, I'll go ahead and merge this PR and then will try to add it to the addon-builder.

@mrstegeman mrstegeman merged commit a278065 into WebThingsIO:master Jan 15, 2020
rzr added a commit to CrossStream/addon-builder that referenced this pull request Jan 15, 2020
Change-Id: I072c5ce6005a27fbc6a8229ed82257eba6ec64c1
Forwarded: https://github.com/mozilla-iot/addon-builder/pulls
Relate-to: WebThingsIO/addon-list#650
Signed-off-by: Philippe Coval <rzr@users.sf.net>
rzr added a commit to CrossStream/addon-builder that referenced this pull request Jan 15, 2020
Change-Id: I996287cec08dc1af2edc7f0bb1fe090027e1e579
Forwarded: https://github.com/mozilla-iot/addon-builder/pulls
Relate-to: WebThingsIO/addon-list#650
Signed-off-by: Philippe Coval <rzr@users.sf.net>
rzr added a commit to CrossStream/addon-builder that referenced this pull request Jan 15, 2020
Change-Id: I072c5ce6005a27fbc6a8229ed82257eba6ec64c1
Forwarded: https://github.com/mozilla-iot/addon-builder/pulls/42
Relate-to: WebThingsIO/addon-list#650
Signed-off-by: Philippe Coval <rzr@users.sf.net>
rzr added a commit to CrossStream/addon-builder that referenced this pull request Jan 15, 2020
Change-Id: I996287cec08dc1af2edc7f0bb1fe090027e1e579
Forwarded: https://github.com/mozilla-iot/addon-builder/pulls/42
Relate-to: WebThingsIO/addon-list#650
Signed-off-by: Philippe Coval <rzr@users.sf.net>
mrstegeman pushed a commit to WebThingsIO/addon-builder that referenced this pull request Jan 15, 2020
* generic-sensors-adapter: Add gitmodule

Change-Id: I072c5ce6005a27fbc6a8229ed82257eba6ec64c1
Forwarded: https://github.com/mozilla-iot/addon-builder/pulls/42
Relate-to: WebThingsIO/addon-list#650
Signed-off-by: Philippe Coval <rzr@users.sf.net>

* generic-sensors-adapter: Add to build-addons.sh

Change-Id: I996287cec08dc1af2edc7f0bb1fe090027e1e579
Forwarded: https://github.com/mozilla-iot/addon-builder/pulls/42
Relate-to: WebThingsIO/addon-list#650
Signed-off-by: Philippe Coval <rzr@users.sf.net>
@rzr
Copy link
Contributor Author

rzr commented May 8, 2020

I just opened a PR, to use builder, but I might have missed other steps, any documentation I should look at:

#823

@mrstegeman
Copy link
Contributor

You can not trigger an actual release build, only I can. So you’ll just have to let me know when you’re ready.

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