Skip to content

Conversation

@ryankurte
Copy link
Contributor

rust-i2cdev has a new minor version that removes a pile of dependencies (and speeds up builds measurably), see: rust-embedded/rust-i2cdev#47

This PR bumps i2cdev to the updated version and adds a missing newline to the end of cargo.toml

@ryankurte ryankurte requested a review from a team as a code owner December 20, 2018 10:46
@ryankurte
Copy link
Contributor Author

bors try

bors bot added a commit that referenced this pull request Dec 20, 2018
@bors
Copy link
Contributor

bors bot commented Dec 20, 2018

try

Build succeeded

Copy link
Contributor

@therealprof therealprof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@therealprof
Copy link
Contributor

bors r+

bors bot added a commit that referenced this pull request Dec 20, 2018
14: Bump i2cdev version r=therealprof a=ryankurte

rust-i2cdev has a new minor version that removes a pile of dependencies (and speeds up builds measurably), see: rust-embedded/rust-i2cdev#47

This PR bumps i2cdev to the updated version and adds a missing newline to the end of cargo.toml

Co-authored-by: Ryan Kurte <ryankurte@gmail.com>
@bors
Copy link
Contributor

bors bot commented Dec 20, 2018

Build succeeded

@bors bors bot merged commit d32ec2f into master Dec 20, 2018
@bors bors bot deleted the i2c-update branch December 20, 2018 10:55
@ryankurte
Copy link
Contributor Author

Thanks @therealprof ^_^ will have a look tomorrow at whether we need anything else for a release.

bors bot added a commit that referenced this pull request Dec 21, 2018
15: Release new patch on crates.io with minimised i2c dependencies r=therealprof a=ryankurte

Following #14 this release contains a newer version of the i2cdev crate that decreases the number of dependencies and thus build time (measured on an rpi3 from 1hr 20 mins to 20 mins).

Co-authored-by: Ryan Kurte <ryankurte@gmail.com>
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