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

i2c_device: adjust for removal of stop=, compatibly #54

Closed
wants to merge 2 commits into from

Conversation

jepler
Copy link
Member

@jepler jepler commented Jul 26, 2020

This version prints (to the serial console) a message about the deprecation. Then we can remove it HERE in a future release.

I can provide a version which just removes it right now, if that's preferable.

Testing performed: with 6.0.0.a2 on an Adafruit Clue, can instantiate the LSM6DS33 object

This version prints (to the serial console) a message about the deprecation.
Then we can remove it *HERE* in a future release.
@jepler jepler linked an issue Jul 26, 2020 that may be closed by this pull request
Copy link
Contributor

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

I think you could just remove it, without the warning. We've already prepped for this, and the library changes make it final.

The print() may not be seen by many people, and it's going to print on every write().

@jerryneedell
Copy link

Tested with CP 6.0.0-alpha2 on metro_m4_express with 2 I2C devices (aht20 and msa301) -- both work normally -- Thank you!

@dhalbert
Copy link
Contributor

Closed in favor of #55.

@dhalbert dhalbert closed this Jul 26, 2020
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.

adafruit_bus_device writeto(..., stop=stop) deprecated
3 participants