-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update to johnny-five v0.11.1 #8
Conversation
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
rwaldrongh-1333 Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Updates board snapshot and serialize tests to account for the new thermometer `freq` property.
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
Signed-off-by: Rick Waldron <waldron.rick@gmail.com>
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, assuming this is just a fast-forward of our copy of johnny-five. If there are areas that involved merge conflicts or other complexities please let me know so I can look more closely.
The only merge conflicts were a few places in package.json and the readme that we customize to make it obvious that this is the Code.org fork of johnny-five, and not the original project.
These are simply changes we rejected in the merge, so they don't show up in the diff here at all. |
👍 |
Update the code-dot-org fork from v0.10.10 to v0.11.1 (diff). Once this is merged and passes CI, I'll release v0.11.1-cdo.0.
The change we're most interested in for Maker Toolkit is that
Thermometer
now has adisable()
method which will let us tear it down cleanly when disconnecting from the board.